Skip to content

Conversation

@nbaws
Copy link
Contributor

@nbaws nbaws commented Oct 22, 2025

Commit Message: aws: match_included_headers feature
Additional Description: Small feature change requested by a customer to allow for positive matching of permitted headers to sign, compared to the existing match_excluded_headers negative match. Also some minor doc readability fixes added while looking at the API.
Risk Level: Low
Testing: Unit
Docs Changes: API documentation
Release Notes: Changelog
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

nbaws added 4 commits October 17, 2025 01:44
Signed-off-by: Nigel Brittain <[email protected]>
Signed-off-by: Nigel Brittain <[email protected]>
Signed-off-by: Nigel Brittain <[email protected]>
Signed-off-by: Nigel Brittain <[email protected]>
@repokitteh-read-only
Copy link

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to (api/envoy/|docs/root/api-docs/).
envoyproxy/api-shepherds assignee is @abeyad
CC @envoyproxy/api-watchers: FYI only for changes made to (api/envoy/|docs/root/api-docs/).

🐱

Caused by: #41639 was opened by nbaws.

see: more, trace.

Signed-off-by: Nigel Brittain <[email protected]>
@nbaws
Copy link
Contributor Author

nbaws commented Oct 22, 2025

@niax

nbaws added 6 commits October 22, 2025 04:52
Signed-off-by: Nigel Brittain <[email protected]>
Signed-off-by: Nigel Brittain <[email protected]>
Signed-off-by: Nigel Brittain <[email protected]>
Signed-off-by: Nigel Brittain <[email protected]>
Signed-off-by: Nigel Brittain <[email protected]>
Signed-off-by: Nigel Brittain <[email protected]>
Copy link
Contributor

@abeyad abeyad left a comment

Choose a reason for hiding this comment

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

/lgtm api

@abeyad
Copy link
Contributor

abeyad commented Oct 22, 2025

Please resolve merge conflicts , thanks!

@abeyad
Copy link
Contributor

abeyad commented Oct 22, 2025

/assign @niax

as a code owner

@repokitteh-read-only
Copy link

@niax cannot be assigned to this issue.

🐱

Caused by: a #41639 (comment) was created by @abeyad.

see: more, trace.

@abeyad
Copy link
Contributor

abeyad commented Oct 22, 2025

/assign @mattklein123

as codeowner

Signed-off-by: Nigel Brittain <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants