Skip to content

Added GET endpoints for activities#43

Closed
guntejsingh27 wants to merge 2 commits intoRedback-Operations:mainfrom
guntejsingh27:feature/get-activity-endpoints
Closed

Added GET endpoints for activities#43
guntejsingh27 wants to merge 2 commits intoRedback-Operations:mainfrom
guntejsingh27:feature/get-activity-endpoints

Conversation

@guntejsingh27
Copy link

Implemented two GET routes: fetch all activities and fetch activity by ID.

@github-actions
Copy link

github-actions bot commented Sep 8, 2025

🔒 Security Scan Results

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

Bandit Scan Results:
-------------------
Run started:2025-09-08 13:21:39.550036

Test results:
	No issues identified.

Code scanned:
	Total lines of code: 1143
	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: 14
		Medium: 0
		High: 0
	Total issues (by confidence):
		Undefined: 0
		Low: 0
		Medium: 0
		High: 14
Files skipped (0):

Dependency Check Results:
-----------------------

No critical security issues detected.

The code has passed all critical security checks.

Copy link

@jd-deakin jd-deakin left a comment

Choose a reason for hiding this comment

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

Please check report.json.

If this information should be public, please message me on teams to explain in more depth what the data is.

At first glance, this appears to be a breach of API keys, and their secrets.
If so, this will need immediate action, including rotating the keys.

Additionally, you should not be shipping a .db file to the repo.

@guntejsingh27
Copy link
Author

guntejsingh27 commented Sep 16, 2025 via email

@guntejsingh27
Copy link
Author

Just to carry on from the email activity.py was the only file I wanted to add for the PR. The other 3 additional files should not have been added.

@github-actions
Copy link

🔒 Security Scan Results

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

Bandit Scan Results:
-------------------
Run started:2025-09-18 15:20:22.587965

Test results:
	No issues identified.

Code scanned:
	Total lines of code: 1263
	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: 14
		Medium: 0
		High: 0
	Total issues (by confidence):
		Undefined: 0
		Low: 0
		Medium: 0
		High: 14
Files skipped (0):

Dependency Check Results:
-----------------------

No critical security issues detected.

The code has passed all critical security checks.

@ben-AI-cybersec
Copy link
Member

changes added in new PR, closing this one

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.

4 participants