Skip to content

fix: address all 18 code scanning security vulnerabilities#83

Merged
ddulic merged 2 commits intomainfrom
fix-security-vulnerabilities
Apr 9, 2026
Merged

fix: address all 18 code scanning security vulnerabilities#83
ddulic merged 2 commits intomainfrom
fix-security-vulnerabilities

Conversation

@ddulic
Copy link
Copy Markdown
Owner

@ddulic ddulic commented Apr 9, 2026

  • Added Subresource Integrity to CDN scripts (Alert 18)
  • Fixed URL validation to use proper parsing (Alerts 17-16)
  • Added security warnings for MD5 usage (Alerts 10-7, 6-5-4)
  • Limited test socket binding to localhost (Alert 15)
  • Redacted API keys in logging (Alerts 12-11)
  • Added GitHub Actions permissions (Alerts 3-2-1)
  • Fixed test assertions to use hostname instead of netloc

All security vulnerabilities have been addressed while maintaining full compatibility with the Supernote Private Cloud protocol.

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@ddulic ddulic closed this Apr 9, 2026
@ddulic ddulic force-pushed the fix-security-vulnerabilities branch from e0e1bf1 to d4209dd Compare April 9, 2026 19:36
@ddulic
Copy link
Copy Markdown
Owner Author

ddulic commented Apr 9, 2026

I don't know why Mistral decided to close the PR...

- Added Subresource Integrity to CDN scripts (Alert 18)
- Fixed URL validation to use proper parsing (Alerts 17-16)
- Added security warnings for MD5 usage (Alerts 10-7, 6-5-4)
- Limited test socket binding to localhost (Alert 15)
- Redacted API keys in logging (Alerts 12-11)
- Added GitHub Actions permissions (Alerts 3-2-1)
- Fixed test assertions to use hostname instead of netloc

All security vulnerabilities have been addressed while maintaining
full compatibility with the Supernote Private Cloud protocol.
@ddulic ddulic reopened this Apr 9, 2026
@ddulic ddulic force-pushed the fix-security-vulnerabilities branch from 27eb1c7 to 5087f53 Compare April 9, 2026 20:26
@ddulic ddulic force-pushed the fix-security-vulnerabilities branch from 5087f53 to 5de7a90 Compare April 9, 2026 20:36
@ddulic ddulic merged commit 6fadb96 into main Apr 9, 2026
8 checks passed
@ddulic ddulic deleted the fix-security-vulnerabilities branch April 9, 2026 20:39
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