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.

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

feat: upgrade to latest keycloak-js adapter 23.0.5 #186

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yaffol
Copy link

@yaffol yaffol commented Jan 31, 2024

This is required to handle the new iss parameter returned from the keycloak server in the redirect response.

This also updates the types to match the deprecation and removal of KeycloakPromise.

This is required to handle the new `iss` parameter returned from the keycloak server in the redirect response.

This also updates the types to match the deprecation and removal of KeycloakPromise.
@LeoQuote LeoQuote mentioned this pull request Mar 7, 2024
@LeoQuote
Copy link

LeoQuote commented Mar 7, 2024

@chrisnruud please merge this and release, it wont take long

@BlackSmith
Copy link

@baltom Can you please look / approve this pull request? This is really big blocker of usable this lib.

@yaffol
Copy link
Author

yaffol commented Jun 26, 2024

It's by no means ideal but if it helps any I've released this under my handle at @pvale/vue-keycloak-js on npm (so I can use it!)

@chrisnruud
Copy link
Contributor

Hi! As mentioned in https://github.com/dsb-norge/vue-keycloak-js/blob/main/CONTRIBUTING.md - this project is sadly tied to whichever version of keycloak is distributed with redhat-sso since that is our use case and bumping past that would severly inhibit our own use of this project. Thanks to @yaffol for releasing a fork with more updated dependencies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants