Skip to content

ci: Allowlist extra endpoints for Dependency Review and CodeQL actions#756

Merged
Pr0methean merged 3 commits into
masterfrom
stepsecurity-allowlist
Mar 29, 2026
Merged

ci: Allowlist extra endpoints for Dependency Review and CodeQL actions#756
Pr0methean merged 3 commits into
masterfrom
stepsecurity-allowlist

Conversation

@Pr0methean
Copy link
Copy Markdown
Member

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@Pr0methean Pr0methean enabled auto-merge March 29, 2026 01:55
Copy link
Copy Markdown
Contributor

@amazon-q-developer amazon-q-developer Bot left a comment

Choose a reason for hiding this comment

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

This PR adds necessary endpoint allowlists for Dependency Review and CodeQL actions, which is a good security practice. However, there are critical configuration errors that will cause the workflows to fail.

Critical Issues Found:

  • Missing port numbers on newly added endpoints in both workflow files
  • The Harden Runner action requires consistent port specification (:443) for all HTTPS endpoints

Required Changes:
All three newly added endpoints need :443 port numbers appended to match the existing endpoint format and ensure the workflows function correctly.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

Comment thread .github/workflows/codeql.yml Outdated
Comment thread .github/workflows/ci.yaml Outdated
Pr0methean and others added 2 commits March 28, 2026 20:30
Co-authored-by: amazon-q-developer[bot] <208079219+amazon-q-developer[bot]@users.noreply.github.com>
Signed-off-by: Chris Hennick <4961925+Pr0methean@users.noreply.github.com>
Co-authored-by: amazon-q-developer[bot] <208079219+amazon-q-developer[bot]@users.noreply.github.com>
Signed-off-by: Chris Hennick <4961925+Pr0methean@users.noreply.github.com>
@Pr0methean Pr0methean added this pull request to the merge queue Mar 29, 2026
Merged via the queue into master with commit 8fb7436 Mar 29, 2026
132 of 133 checks passed
@Pr0methean Pr0methean deleted the stepsecurity-allowlist branch March 29, 2026 09:01
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.

1 participant