Skip to content

feat(rest): add a SigV4 auth manager and session - #3092

Open
plusplusjiajia wants to merge 1 commit into
apache:mainfrom
plusplusjiajia:feat/rest-sigv4-session
Open

plusplusjiajia wants to merge 1 commit into
apache:mainfrom
plusplusjiajia:feat/rest-sigv4-session

Conversation

@plusplusjiajia

@plusplusjiajia plusplusjiajia commented Aug 28, 2026 •

Copy link
Copy Markdown
Member

Which issue does this PR close?

Follows #3082. Catalog property selection is handled separately in #2660.

What changes are included in this PR?

Adds SigV4AuthManager and SigV4Session for signing on top of delegate authentication. Applications inject the manager with with_auth_manager; credentials are resolved per request, with signing overrides supported for property-built contextual sessions.

Configured headers are applied before authentication, and the default HTTP client disables redirects for signing managers. from_properties accepts static credentials; applications can inject a credentials provider for other sources.

Are these changes tested?

Unit tests cover credential resolution, delegation, context overrides and error paths. Mock-server tests cover injected managers, redirect behavior and configured Authorization headers.

@plusplusjiajia
plusplusjiajia force-pushed the feat/rest-sigv4-session branch from a98b276 to 0bddb73 Compare September 2, 2026 07:03
@plusplusjiajia
plusplusjiajia force-pushed the feat/rest-sigv4-session branch 3 times, most recently from 8926082 to c692cdc Compare September 22, 2026 08:59
@plusplusjiajia
plusplusjiajia force-pushed the feat/rest-sigv4-session branch 3 times, most recently from 64ef539 to 37476d0 Compare October 2, 2026 12:21
@plusplusjiajia
plusplusjiajia force-pushed the feat/rest-sigv4-session branch 5 times, most recently from 2c53b79 to 7cd7af2 Compare October 10, 2026 02:31
@plusplusjiajia plusplusjiajia changed the title feat(rest): add a SigV4 auth manager over a delegate session feat(rest): add a SigV4 auth manager and session Oct 10, 2026
@plusplusjiajia
plusplusjiajia marked this pull request as ready for review October 10, 2026 02:45
@plusplusjiajia
plusplusjiajia force-pushed the feat/rest-sigv4-session branch from 7cd7af2 to 3b9a31a Compare October 10, 2026 03:00
@plusplusjiajia
plusplusjiajia force-pushed the feat/rest-sigv4-session branch from 3b9a31a to 802a29f Compare October 19, 2026 03:17

This branch has not been deployed

No deployments
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