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

okta-signin-widget 7.0+ No longer references cookies #31

Open
eburkland opened this issue Mar 29, 2023 · 3 comments
Open

okta-signin-widget 7.0+ No longer references cookies #31

eburkland opened this issue Mar 29, 2023 · 3 comments

Comments

@eburkland
Copy link

cookie_state = request.COOKIES["okta-oauth-state"]

Trying to implement the latest okta-signin-widget into my existing django project. I'm having difficulty with the callback method. Curernt okta-signin-widget as of 7.0+ no longer creates state or nonce cookies. Also it doesn't look like this supports PKCE.

There seems to be a number of github repositories that reference okta and django but none of them work very well with the current standards. Any ideas?
Thanks!

@eu2pey4
Copy link

eu2pey4 commented Mar 30, 2023

For me, I had to settle on just using the Okta Sign-in Widget 5.0.1

@eburkland
Copy link
Author

When I exchange the code for tokens by calling the /token endpoint I get the access token, id token as expected. If I attempt to use the same code twice it doesn't work. This to me is a good thing as you wouldn't want someone to keep trying to use the same code to get tokens. Because of this I'm wondering if it is necessary to do checks on state and nonce. Just a thought.

@silver8ack
Copy link

Something seems broken with the middleware of this project. I'm not sure what it is. See issue 33 I created.

I can get authentication working using the /accounts/login url as long as I don't have the middleware enabled. It seems like the middleware is removing the session or something. I never see a session get created in the database when the middleware is enabled.

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

No branches or pull requests

3 participants