search

ueberauth / ueberauth_facebook

Facebook OAuth2 Strategy for Überauth.
MIT License
77 stars 64 forks source link

Loosen ueberauth dependency #64

Closed arjan closed 3 years ago

arjan commented 3 years ago

This ensures that we can use 0.7, which has been released

yordis commented 3 years ago

@arjan yeah, we need to make sure that does actually work with v0.7, have you test it?

arjan commented 3 years ago

You are right, there is indeed an issue in the auth callback:

FAIL: %Ueberauth.Failure{errors: [%Ueberauth.Failure.Error{message: "Cross-Site Request Forgery attack", message_key: :csrf_attack}], provider: :facebook, strategy: Ueberauth.Strategy.Facebook}

I will investigate.

arjan commented 3 years ago

It does not work indeed, due to the changes in https://github.com/ueberauth/ueberauth/issues/135

Will create a new PR