Skip to content

fix: OAuth error #934

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

Merged

Conversation

nikhilsinhaparseable
Copy link
Contributor

issue: already borrowed: BorrowMutError when try login with oauth because of req.connection_info()
which makes an immutable borrow of req method extract_session_key_from_req() also borrows req
because of which the server panics with the error

fix is to use connection_info().clone() to avoid multiple borrows

issue: already borrowed: BorrowMutError when try login with oauth
because of req.connection_info() which makes an immutable borrow of req method extract_session_key_from_req() also borrows req because of which the server panics with the error
fix is to use connection_info().clone() to avoid multiple borrows
@nitisht nitisht merged commit a5468b0 into parseablehq:main Sep 23, 2024
8 checks passed
parmesant pushed a commit to parmesant/parseable that referenced this pull request Sep 26, 2024
issue: already borrowed: BorrowMutError when try login with oauth
because of req.connection_info() which makes an immutable borrow of 
req method extract_session_key_from_req() also borrows req because of which 
the server panics with the error fix is to use connection_info().clone() to 
avoid multiple borrows
@nikhilsinhaparseable nikhilsinhaparseable deleted the oidc-login-failure-fix branch January 18, 2025 18:45
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 this pull request may close these issues.

2 participants