Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature/token exchange rebase #575

Closed
wants to merge 26 commits into from
Closed

Feature/token exchange rebase #575

wants to merge 26 commits into from

Conversation

rezaansyed
Copy link
Contributor

WHY are these changes introduced?

Fixes #0000

WHAT is this pull request doing?

Type of change

  • Patch: Bug (non-breaking change which fixes an issue)
  • Minor: New feature (non-breaking change which adds functionality)
  • Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

  • I have used yarn changeset to create a draft changelog entry (do NOT update the CHANGELOG.md files manually)
  • I have added/updated tests for this change
  • I have documented new APIs/updated the documentation for modified APIs (for public APIs)

rachel-carvalho and others added 23 commits January 15, 2024 09:20
@rachel-carvalho @rezaansyed
add unstable_newEmbeddedAuthStrateg future flag
9a644a9
@rachel-carvalho @rezaansyed
add token exchange strategy
82da3dc
@rezaansyed
Enable token exchange strategy when future flag is enabled
f32195f
@rachel-carvalho @rezaansyed
setup auth code flow tests with future flag off
65f8431
@rachel-carvalho @rezaansyed
add tests to token exchange strategy
9b4ace8
@rachel-carvalho @rezaansyed
move tests that redirect when no session is found into auth code flow…
e494016 
… test folder

move embedded search param check to generic doc request tests
@rezaansyed
throw 500 for most token exchange errors and complete code coverage
b79c046
@rezaansyed
Fix logging
db3fba0
@rezaansyed
Update types to account for unstable_tokenExchange and unstable_newEm…
efa1729 
…beddedAuthStrategy future flags
@rezaansyed
Add changeset
915e2a4
@rezaansyed
Update future flag description
ba0dbf8
@rezaansyed
Log throttling errors on webhook registration
7888756
@rezaansyed
Refactor to address review comments
30aafff
@rezaansyed
Introduce idempotent promise handler
691f223
@zzooeeyy @rezaansyed
Redirect to install from app home page if future flag is on and app i…
bf7d9fd 
…s embedded
@zzooeeyy @rezaansyed
Redirect to new admin URL instead of legacy
c750f3e
@rezaansyed
Clear stale identifiers in IdempotentPromiseHandler
a9d73c8
@rachel-carvalho @rezaansyed
move invalid access token redirect to auth tests to auth code flow st…
eb773d9 
…rategy
@rachel-carvalho @rezaansyed
delegate handling invalid access token errors to auth strategy
3d7f445
@rezaansyed
Add error handling for token exchange strategy on invalid API requests
97cc308
@rezaansyed
Move SessionContext to authenticate as it is only used there
e589e80
@rezaansyed
Inject error handling strategy on after auth API calls
9f02294
@rezaansyed
Pass in handleClientError method from the strategy
d00eb03
@rezaansyed rezaansyed force-pushed the feature/token-exchange-rebase branch from 8db3e25 to d00eb03 Compare January 15, 2024 14:20
@rezaansyed rezaansyed force-pushed the feature/token-exchange-rebase branch from e274fbe to c4fb5fb Compare January 17, 2024 16:41
@rezaansyed rezaansyed closed this Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@rezaansyed @rachel-carvalho @zzooeeyy @paulomarg