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

Client receives code verifier error when the user has never logged in before #320

Closed
jnweiger opened this issue Nov 17, 2021 · 3 comments
Closed

Comments

@jnweiger
Copy link
Contributor

Seen with oauth2-0.5.2 on ownCloud 10.8.1 prealpha (daily) Build:2021-11-16T22:03:07+00:00 3e5ae8e8a76638d92d0167dfa3ea306c41d1bf08

  • User admin creates user01 with password user01
  • Desktop client 2.9.1 connects, -> the oauth flow starts, user admin is suggested.
  • Switch user, log in user01, authorize. an error occurs:
    image

On a second attempt, the client can authorize just fine.

@jnweiger jnweiger mentioned this issue Nov 17, 2021
44 tasks
@jnweiger
Copy link
Contributor Author

jnweiger commented Nov 17, 2021

Similar to #309 - likely a duplicate... -- if so, the issue persists in 0.5.2

@jnweiger
Copy link
Contributor Author

Reproduced with core 10.9.0-beta1 and client 2.9.2-rc5 and oauth2-0.5.2

@jvillafanez
Copy link
Member

Similar to #309 - likely a duplicate... -- if so, the issue persists in 0.5.2

Same thoughts. It seems related to the "user switch" feature.

As said in the linked ticket, I think it's better to change the button to a "logout" button and let the user restart the process from the client.

Assuming the client doesn't have a browser with a valid ownCloud session, the flow seems to work fairly well even if the user hasn't logged in yet. The only problem is that the "authorization-successful" page should be public. Currently, that page requires the user to be logged in, which is a bit weird.
It seems the flow doesn't login the user / doesn't create a valid session, so ownCloud requires login to access to the "authorization-successful" page even though the client already has a valid token and can access ownCloud from that point.

@jnweiger jnweiger mentioned this issue Mar 11, 2022
42 tasks
@IljaN IljaN closed this as completed Apr 8, 2024
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