Skip to content

Refactor authorizer info in API Gateway invocation context#7740

Merged
whummer merged 2 commits intomasterfrom
apigw-auth-claims
Feb 24, 2023
Merged

Refactor authorizer info in API Gateway invocation context#7740
whummer merged 2 commits intomasterfrom
apigw-auth-claims

Conversation

@whummer
Copy link
Member

@whummer whummer commented Feb 23, 2023

Refactor authorizer info in API Gateway invocation context. Companion to upstream PR.

The main changes in this PR is to rename the auth_info attribute and unify it with auth_context in the API GW invocation context

The PR also adds a few type hints across the codebase.

@whummer whummer requested a review from calvernaz as a code owner February 23, 2023 18:16
@whummer whummer temporarily deployed to localstack-ext-tests February 23, 2023 18:16 — with GitHub Actions Inactive
@whummer whummer temporarily deployed to localstack-ext-tests February 23, 2023 18:16 — with GitHub Actions Inactive
@github-actions
Copy link

github-actions bot commented Feb 23, 2023

LocalStack integration with Pro

       3 files  ±0         3 suites  ±0   1h 30m 1s ⏱️ - 7m 58s
1 753 tests ±0  1 385 ✔️ +1  368 💤  - 1  0 ±0 
2 471 runs  ±0  1 761 ✔️ +1  710 💤  - 1  0 ±0 

Results for commit bf62128. ± Comparison against base commit 4be5754.

♻️ This comment has been updated with latest results.

@whummer whummer temporarily deployed to localstack-ext-tests February 23, 2023 21:09 — with GitHub Actions Inactive
Copy link
Contributor

@calvernaz calvernaz left a comment

Choose a reason for hiding this comment

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

Thanks for the speedy reaction and the cleanup!

@whummer whummer temporarily deployed to localstack-ext-tests February 24, 2023 01:12 — with GitHub Actions Inactive
@coveralls
Copy link

Coverage Status

Coverage: 85.061% (+0.005%) from 85.056% when pulling bf62128 on apigw-auth-claims into 4be5754 on master.

@whummer whummer merged commit b1cf40f into master Feb 24, 2023
@whummer whummer deleted the apigw-auth-claims branch February 24, 2023 14:02
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.

3 participants