Skip to content

Issuer caching - #33

Merged
langsamu merged 4 commits into
mainfrom
issuer-caching
Aug 24, 2026
Merged

Issuer caching#33
langsamu merged 4 commits into
mainfrom
issuer-caching

Conversation

@langsamu

@langsamu langsamu commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Similar to #30, this change introduces caching of IdPs (OAuth Authorization Servers).

First, the previous GetIssuerCallback, a lambda signature type, is converted to the interface IssuerProvider to make it easier to add state to its instances.

Then a CachingIssuerProvider is introduced to overlay atop an existing provider. It is used in the demo page:

const issuerUi = document.querySelector("idp-picker")
const issuerProvider = new CachingIssuerProvider(issuerUi)
const callbackUri = new URL("/callback.html", location.href).toString()
const dPoPTokenProvider = new DPoPTokenProvider(callbackUri, ui.getCode.bind(ui), issuerProvider)

The cache is again internal to the caching provider instead of being taken from the caller. I will address this across the board later.

@langsamu
langsamu marked this pull request as draft August 20, 2026 09:02
@langsamu langsamu changed the title issuer caching Issuer caching Aug 20, 2026
@langsamu
langsamu marked this pull request as ready for review August 20, 2026 09:11
@langsamu

Copy link
Copy Markdown
Collaborator Author

@copilot fix merge conflicts in this pull request.

Co-authored-by: langsamu <240741+langsamu@users.noreply.github.com>

Copilot AI commented Aug 24, 2026

Copy link
Copy Markdown

@copilot fix merge conflicts in this pull request.

Resolved in commit 8d7b7ec by merging latest main into this branch and fixing the conflict in src/DPoPTokenProvider.ts.

@langsamu
langsamu merged commit 8ad57fc into main Aug 24, 2026
5 checks passed
@langsamu
langsamu deleted the issuer-caching branch August 24, 2026 07:45
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.

2 participants