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

Refresh token XHR Request #2188

Closed
romilin opened this issue May 22, 2023 · 4 comments · Fixed by #2223
Closed

Refresh token XHR Request #2188

romilin opened this issue May 22, 2023 · 4 comments · Fixed by #2223
Assignees

Comments

@romilin
Copy link

romilin commented May 22, 2023

Hello @EricWittmann ,

The refresh token request failed with this logs on chrome :
token-apicurio-live

The problem also exist on firefox, here is the logs i get :
firefox-token

I also have the problem when i use my local apicurio. It appears at the end of the token life.
I tried to changes some settings on my keycloak but i can't solve the problem.

Do you have any idea on how to correct that ?

@romilin
Copy link
Author

romilin commented May 24, 2023

Hello,

I think the solution to this problem is not a keycloak settings.

The "token" request doesn't have a preflight request such as "designs" request. Maybe adding this preflight for token will correct that bug ?

What do you think about that ?

@romilin
Copy link
Author

romilin commented Jun 1, 2023

Hello,

An update on this subject :
I changed the token request to POST instead of GET and I got this :
image

As u can see, the "Access-Control-Allow-Origin" is in the response headers.

If I'm right, Quarkus Code Flow Redirect is use in the next request to keycloak. I tried to change this request to a POST too but still don't know how to do it.
Here is the response to the preflight request without the headers I need :
image

Any idea on how to make this "quarkus" request to do a POST instead of GET ?

@romilin
Copy link
Author

romilin commented Jun 21, 2023

Hello @EricWittmann
Still no idea on how to correct that bug 😞

@carlesarnal : I saw your work in QuarkusAuthenticationFilter
The solution might be there ?

i just tried the live version of Apicurio, the bug is not corrected so users can't work more than 5 minutes (acces token lifespan duration in keycloak) without manual refresh.

Any ideas ?

@carlesarnal
Copy link
Member

@EricWittmann we should release a new version of Studio with this fix to the upstream version.

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 a pull request may close this issue.

2 participants