Replies: 1 comment 1 reply
|
Right now we don't have the Codex connector or even Copilot set up to automatically review PRs in the API repos. A possible mitigating step could be to configure that with explicit instructions to highlight any changes which touch secrets or the legacy/PII databases. We could also look into custom socket bots that examine function dependency or authorizer changes. This would make it easier to efficiently review PRs and keep collaboration default-open. I've tried to minimize the reviewability burden and injection risk by keeping the functions and endpoints as small and simple as possible. I have also actively axed operating handlers in the past which exceeded a reasonable couple hundred lines of code. |
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Access, managing IAM workflows, and documentation are all on GitHub. They are the production behavior of our systems and can be amended through pull requests.
A critical example of this attack is changing a public API endpoint from its Gateway Lambda handler to cease using its team authorizer. That, or just return a new function call which returns information the handler processes but does not return.
It should be easy to see when this happens, since it'd need to be in the return stream, presuming call logging stays unpublished. But there's the risk of subtle weakness injection by modifying complex arithmetic like hashing, which could hide rainbow-tree-able customer information.
All reactions