soofstad / react-oauth2-pkce

Provider agnostic OAuth2 Authorization Code flow with PKCE for React
MIT License
125 stars 53 forks source link

Bug: codeVerifier is not set in sessionStorage (sometimes) #159

Closed bondarev123 closed 3 months ago

bondarev123 commented 5 months ago

As I say, it never happened when login page was on another port.

Do you have any suggestions?

Steps To Reproduce

The current behavior

PKCE_code_verifier is sometimes not being set in session storage

The expected behavior

PKCE_code_verifier is always being set in session storage

sebastianvitterso commented 5 months ago

I can't say that I see any reason why this would happen, and certainly not depending on the port number used. Is that the only thing you've changed since getting these bugs?

bondarev123 commented 5 months ago

Seems like a JS sessionStorage's problem. When changed from sessionStorage to localStorage problem dissapeared.

sebastianvitterso commented 5 months ago

Ok, well it's good that the issue was resolved, but I'm curious as to what was the actual problem. If you'd like to do more research into the issue, go ahead, otherwise feel free to close the issue.

github-actions[bot] commented 3 months ago

Stale issue message