Skip to content

Conversation

@Kados47
Copy link
Contributor

@Kados47 Kados47 commented Sep 17, 2025

What are the changes?

  • The changes that i have made within this pr is that after uploading all of the datasets within this repo to the Data Warehouse I have then gone through and deleted each of the files uploaded and also any duplicates, I have also made a few minor changes in the documentaion to reflect this.

Who worked on the changes?

  • Just myself: Kadyn Trayford (s223308953)

New Components/Features

  • Deleted Datasets

Enhanced Components/Features

  • Edited readme files to reflect the changes

Other Changes/Fixes

  • deleted duplicate files.

Dataset Uploaded to Warehouse, and now deleted
…pload_randomized.csv

Uploaded to warehouse and now deleteted
…tion_dataset.csv

Uploaded to warehouse and now deleteted
…es_012_health_indicators_BRFSS2015.csv

Uploaded to Data Warehouse and now deleted
…es_binary_5050split_health_indicators_BRFSS2015.csv

Uploaded to Data Warehouse and now deleted
…es_binary_health_indicators_BRFSS2015.csv

Uploaded to Data Warehouse and now deleted
Uploaded to Data Warehouse and now deleted
…diction_dataset.csv

Uploaded to Data Warehouse and now deleted
…ataset.csv

Uploaded to Data Warehouse and now deleted
…ATASET.csv

Uploaded to Data Warehouse and now deleted
Uploaded to Data Warehouse and now deleted
Uploaded to Data Warehouse and now deleted
…ets directory

Uploaded to Data Warehouse and now deleted
Updating Documentaion
Updated Documentation
@Kados47 Kados47 requested review from a team as code owners September 17, 2025 12:15
@github-actions
Copy link

🔒 Security Scan Results

🔒 Security Scan Results
=========================

Bandit Scan Results:
-------------------
Run started:2025-09-17 12:16:45.921245

Test results:
>> Issue: [B113:request_without_timeout] Call to requests without timeout
   Severity: Medium   Confidence: Low
   CWE: CWE-400 (https://cwe.mitre.org/data/definitions/400.html)
   More Info: https://bandit.readthedocs.io/en/1.8.6/plugins/b113_request_without_timeout.html
   Location: ./data_science/voice_assistant/generate_response.py:19:10
18	def get_request(query):
19	    req = requests.get(f'https://www.google.com/search?q={query}')
20	

--------------------------------------------------
>> Issue: [B404:blacklist] Consider possible security implications associated with the subprocess module.
   Severity: Low   Confidence: High
   CWE: CWE-78 (https://cwe.mitre.org/data/definitions/78.html)
   More Info: https://bandit.readthedocs.io/en/1.8.6/blacklists/blacklist_imports.html#b404-import-subprocess
   Location: ./data_science/voice_assistant/text_to_speech.py:5:0
4	import os
5	import subprocess
6	# Function to generate a response using our NLP model

--------------------------------------------------
>> Issue: [B607:start_process_with_partial_path] Starting a process with a partial executable path
   Severity: Low   Confidence: High
   CWE: CWE-78 (https://cwe.mitre.org/data/definitions/78.html)
   More Info: https://bandit.readthedocs.io/en/1.8.6/plugins/b607_start_process_with_partial_path.html
   Location: ./data_science/voice_assistant/text_to_speech.py:15:4
14	    time.sleep(2)
15	    subprocess.run(['say', response], check=True)
16	

--------------------------------------------------
>> Issue: [B603:subprocess_without_shell_equals_true] subprocess call - check for execution of untrusted input.
   Severity: Low   Confidence: High
   CWE: CWE-78 (https://cwe.mitre.org/data/definitions/78.html)
   More Info: https://bandit.readthedocs.io/en/1.8.6/plugins/b603_subprocess_without_shell_equals_true.html
   Location: ./data_science/voice_assistant/text_to_speech.py:15:4
14	    time.sleep(2)
15	    subprocess.run(['say', response], check=True)
16	

--------------------------------------------------

Code scanned:
	Total lines of code: 991
	Total lines skipped (#nosec): 0
	Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 0

Run metrics:
	Total issues (by severity):
		Undefined: 0
		Low: 3
		Medium: 1
		High: 0
	Total issues (by confidence):
		Undefined: 0
		Low: 1
		Medium: 0
		High: 3
Files skipped (0):

No critical security issues detected.

The code has passed all critical security checks.

@lperry022 lperry022 self-assigned this Sep 22, 2025
Copy link
Contributor

@lperry022 lperry022 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lperry022 lperry022 merged commit 8d14d9b into Redback-Operations:main Sep 22, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants