-
Notifications
You must be signed in to change notification settings - Fork 3
feat(journey-client): wellknown-endpoint-config-support #525
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.
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
base: main
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: 73386a2 The changes in this PR will be included in the next version bump. This PR includes changesets to release 12 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
📝 WalkthroughWalkthroughThis PR introduces OIDC well-known endpoint discovery support to the Changes
Sequence DiagramsequenceDiagram
participant App as App
participant JourneyFactory as Journey Factory
participant ConfigResolver as Config Resolver
participant WellknownAPI as Wellknown API
participant Redux as Redux Store
participant JourneyStore as Journey Store
rect rgba(100, 150, 200, 0.5)
Note over App,JourneyStore: Well-known Configuration Path
App->>JourneyFactory: journey(JourneyConfigInput with wellknown URL)
JourneyFactory->>ConfigResolver: hasWellknownConfig(config)?
ConfigResolver-->>JourneyFactory: true
JourneyFactory->>ConfigResolver: resolveAsyncConfig(config)
ConfigResolver->>WellknownAPI: Fetch well-known configuration
WellknownAPI->>Redux: Cache result
WellknownAPI-->>ConfigResolver: WellKnownResponse
ConfigResolver->>ConfigResolver: inferRealmFromIssuer(issuer)
ConfigResolver->>ConfigResolver: Merge configurations
ConfigResolver-->>JourneyFactory: InternalJourneyClientConfig
end
rect rgba(150, 100, 200, 0.5)
Note over App,JourneyStore: Standard Configuration Path
App->>JourneyFactory: journey(JourneyConfigInput)
JourneyFactory->>ConfigResolver: hasWellknownConfig(config)?
ConfigResolver-->>JourneyFactory: false
JourneyFactory->>ConfigResolver: normalizeConfig(config)
ConfigResolver-->>JourneyFactory: InternalJourneyClientConfig
end
JourneyFactory->>JourneyStore: createJourneyStore(resolvedConfig)
JourneyStore->>Redux: Register API, reducer, middleware
Redux-->>JourneyStore: Store created
JourneyStore-->>App: Store ready
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
View your CI Pipeline Execution ↗ for commit 73386a2
☁️ Nx Cloud last updated this comment at |
@forgerock/davinci-client
@forgerock/device-client
@forgerock/journey-client
@forgerock/oidc-client
@forgerock/protect
@forgerock/sdk-types
@forgerock/sdk-utilities
@forgerock/iframe-manager
@forgerock/sdk-logger
@forgerock/sdk-oidc
@forgerock/sdk-request-middleware
@forgerock/storage
commit: |
Codecov Report❌ Patch coverage is ❌ Your project status has failed because the head coverage (19.50%) is below the target coverage (40.00%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #525 +/- ##
==========================================
+ Coverage 18.79% 19.50% +0.71%
==========================================
Files 140 144 +4
Lines 27640 28054 +414
Branches 980 1010 +30
==========================================
+ Hits 5195 5473 +278
- Misses 22445 22581 +136
🚀 New features to boost your workflow:
|
|
Deployed 04794f6 to https://ForgeRock.github.io/ping-javascript-sdk/pr-525/04794f6a6212c98698f331ca125dc65582860a6d branch gh-pages in ForgeRock/ping-javascript-sdk |
📦 Bundle Size Analysis📦 Bundle Size Analysis🚨 Significant Changes🔻 @forgerock/journey-client - 0.0 KB (-82.4 KB, -100.0%) ➖ No Changes➖ @forgerock/device-client - 9.2 KB 13 packages analyzed • Baseline from latest Legend🆕 New package ℹ️ How bundle sizes are calculated
🔄 Updated automatically on each push to this PR |
JIRA Ticket
https://pingidentity.atlassian.net/browse/SDKS-4665
Description
Add support for well known endpoint
Summary by CodeRabbit
Release Notes
New Features
Tests
✏️ Tip: You can customize this high-level summary in your review settings.