Skip to content

SCA Finding fixed - #423

Merged
SB-rohandhiman merged 1 commit into
nextfrom
PLAT-16740-SCA-FINDING-FIXED
Aug 13, 2026
Merged

SCA Finding fixed#423
SB-rohandhiman merged 1 commit into
nextfrom
PLAT-16740-SCA-FINDING-FIXED

Conversation

@SB-rohandhiman

Copy link
Copy Markdown

Fix SAST/secret findings: GitHub Actions permissions and Django SECRET_KEY

Addresses PLAT-16976, PLAT-16740, and PLAT-16763.

GitHub Actions (maze-runner.yml, python-package.yml)

  • Added permissions: contents: read at top level
  • Added permissions: contents: none on test-done job
  • Follows least-privilege principle for workflow tokens

Django SECRET_KEY — example files (3 files)

  • Replaced hardcoded keys with os.environ.get('DJANGO_SECRET_KEY', 'django-insecure-testonly')
  • Falls back to a clearly marked insecure test key

Django SECRET_KEY — test fixtures (5 files)

  • Replaced random-looking keys with django-insecure-testonly
  • No env var needed for test fixtures

Changelog

  • Added PR reference for level parameter enhancement

@SB-PawanN SB-PawanN left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@SB-rohandhiman
SB-rohandhiman marked this pull request as ready for review August 13, 2026 09:21
@SB-rohandhiman
SB-rohandhiman merged commit c5dd5d1 into next Aug 13, 2026
66 checks passed
@SB-rohandhiman
SB-rohandhiman deleted the PLAT-16740-SCA-FINDING-FIXED branch August 13, 2026 09:23
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.

3 participants