Skip to content
This repository has been archived by the owner on Oct 5, 2023. It is now read-only.

[Security] Support for the :state parameter required with omniauth-oauth2 v1.1 #43

Open
ZenCocoon opened this issue Jul 10, 2012 · 3 comments

Comments

@ZenCocoon
Copy link

Hi,

A few days ago, a security update has been made to omniauth-oauth2, this one uses the :state parameter to mitigate CSRF. omniauth/omniauth-oauth2#18

However, it seem that devise_oauth2_providable doesn't handle this parameter as expected as it doesn't send it back in the callback.

A similar issue seems to be faced at omniauth/omniauth-oauth2#20

@adampope
Copy link

+1 to this. Would love to see a fix.

@karlfreeman
Copy link
Contributor

👍

@ZenCocoon
Copy link
Author

Anyone having time to make a pull request?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants