oauth2 fixes #177

Merged
floatingghost merged 2 commits from oauth2-fixes into develop 2022-08-21 14:46:53 +00:00

a lot of the oauth2 weirdness in pleroma was caused by it using the cookie-based session to pull out tokens - this meant that if you were to change frontends (and hence your token), your cookie would change as well, making the backend think you were a new user and skipping any checks for existing authorization

this change more or less removes session store for oauth2 (and honestly we should remove a lot more session stuff, oauth2 doesn't need state as well...)

a lot of the oauth2 weirdness in pleroma was caused by it using the cookie-based session to pull out tokens - this meant that if you were to change frontends (and hence your token), your cookie would change as well, making the backend think you were a new user and skipping any checks for existing authorization this change more or less removes session store for oauth2 (and honestly we should remove a lot more session stuff, oauth2 doesn't need state as well...)
Do not use session for oauth tokens
Some checks are pending
ci/woodpecker/push/woodpecker Pipeline is pending
d5004b02b4
totally ignore session when handling oauth2
Some checks are pending
ci/woodpecker/push/woodpecker Pipeline is pending
ci/woodpecker/pr/woodpecker Pipeline was successful
4b5e614d9a
floatingghost deleted branch oauth2-fixes 2022-08-21 14:46:53 +00:00
floatingghost referenced this pull request from a commit 2022-08-21 14:46:53 +00:00
floatingghost referenced this pull request from a commit 2022-08-21 15:22:20 +00:00
rat referenced this pull request from a commit 2024-02-25 20:33:29 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AkkomaGang/akkoma!177
No description provided.