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

Debug-log cookie login token regeneration #33930

Closed

Conversation

ChristophWurst
Copy link
Member

Follow-up to #33772 now that we have our first real-world feedback in #33919.

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst ChristophWurst added the 3. to review Waiting for reviews label Sep 7, 2022
@ChristophWurst ChristophWurst added this to the Nextcloud 25 milestone Sep 7, 2022
@ChristophWurst ChristophWurst self-assigned this Sep 7, 2022
This was referenced Sep 7, 2022
@ChristophWurst ChristophWurst mentioned this pull request Sep 12, 2022
8 tasks
@skjnldsv skjnldsv mentioned this pull request Sep 15, 2022
This was referenced Sep 20, 2022
@blizzz blizzz modified the milestones: Nextcloud 25, Nextcloud 26 Sep 22, 2022
@ChristophWurst ChristophWurst removed this from the Nextcloud 26 milestone Oct 19, 2022
@nickvergessen
Copy link
Member

Seems to confirm:

{"reqId":"4Q9o5Oo77xzHnOYfQIUH","level":2,"time":"2022-10-20T08:02:53+00:00","remoteAddr":"","user":"--","app":"core","method":"GET","url":"/ocs/v2.php/apps/spreed/api/v1/chat/…","message":"Regenerating cookie login token for … after successful verification","userAgent":"Mozilla/5.0 …","version":"25.0.0.18","data":{"app":"core"}}
{"reqId":"o2QPBCrQftd2z0fxSojG","level":3,"time":"2022-10-20T08:02:53+00:00","remoteAddr":"","user":"--","app":"core","method":"GET","url":"/ocs/v2.php/apps/notifications/api/v2/notifications","message":"Tried to log in … but could not verify token","userAgent":"Mozilla/5.0 …","version":"25.0.0.18","data":{"app":"core"}}
{"reqId":"kjn3gxMa96CHx5y1Cp6R","level":3,"time":"2022-10-20T08:02:53+00:00","remoteAddr":"","user":"--","app":"core","method":"GET","url":"/csrftoken","message":"Tried to log in … but could not verify token","userAgent":"Mozilla/5.0 …","version":"25.0.0.18","data":{"app":"core"}}
{"reqId":"hDdnqKnVcQgoLq3Oof4i","level":3,"time":"2022-10-20T08:02:53+00:00","remoteAddr":"","user":"--","app":"core","method":"GET","url":"/csrftoken","message":"Tried to log in … but could not verify token","userAgent":"Mozilla/5.0 …","version":"25.0.0.18","data":{"app":"core"}}
{"reqId":"suSulTRF6QiN7HdiLBr9","level":3,"time":"2022-10-20T08:02:53+00:00","remoteAddr":"","user":"--","app":"core","method":"GET","url":"/csrftoken","message":"Tried to log in … but could not verify token","userAgent":"Mozilla/5.0 …","version":"25.0.0.18","data":{"app":"core"}}

5 parallel requests of the same user, the first one wins the others fail to renew

@ChristophWurst
Copy link
Member Author

This didn't give clear insights. Closing.

@ChristophWurst ChristophWurst deleted the debug/log-cookie-login-token-regeneration branch January 18, 2023 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
Development

Successfully merging this pull request may close these issues.

3 participants