rusackas opened a new pull request, #44038:
URL: https://github.com/apache/superset/pull/44038

   ### SUMMARY
   zizmor's `self-repository` audit flags the `Setup Postgres` step in 
`superset-python-integrationtest.yml` for using `uses: 
./.github/actions/cached-dependencies` instead of the dedicated `$/...` 
self-repository syntax.
   
   `cached-dependencies` is a git submodule (see `.gitmodules`), not a plain 
in-repo directory. GitHub's `$/` self-repository syntax resolves action files 
directly from the repository tree without a submodule-aware checkout, so it 
cannot see into a submodule's gitlink — only the workspace-relative `./` form 
works here. This is a known false positive, and the codebase already suppresses 
it with the same justification everywhere else this action is referenced (e.g. 
`superset-python-presto-hive.yml`, `superset-translations.yml`, and sibling 
steps in this same workflow file).
   
   This PR adds the matching scoped suppression comment for this one 
occurrence, resolving the alert without changing behavior.
   
   Resolves code-scanning alert #2657: 
https://github.com/apache/superset/security/code-scanning/2657
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   N/A (CI workflow comment-only change)
   
   ### TESTING INSTRUCTIONS
   - `pre-commit run --files 
.github/workflows/superset-python-integrationtest.yml` passes (including the 
local `zizmor` hook)
   - No functional change to the workflow; `test-postgres` CI job continues to 
run as before
   
   ### ADDITIONAL INFORMATION
   - [ ] Has associated issue:
   - [ ] Required feature flags:
   - [ ] Changes UI
   - [ ] Includes DB Migration (follow approval process in 
[SIP-59](https://github.com/apache/superset/issues/13351))
     - [ ] Migration is atomic, supports rollback & is backwards-compatible
     - [ ] Confirm DB migration upgrade and downgrade tested
     - [ ] Runtime estimates and downtime expectations provided
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to