search

pow-auth / pow_assent

Multi-provider authentication for your Pow enabled app
https://powauth.com
MIT License
321 stars 50 forks source link

Pow Assent API example incorrect #193

Closed Lazarus404 closed 3 years ago

Lazarus404 commented 3 years ago

There seems to be errors in the API modules. The authorization_controller.ex file uses a redirect_uri function invocation passing it two parameters, yet the implementation of that function requires only conn.

When running the tests, the callback/2 test fails, because the returns JSON has both token and renew_token set to nil.

Can this be updated so I can ensure I fix whatever is missing in my trial?

danschultzer commented 3 years ago

Thanks, opened a PR to resolve it. This updated version should work with the newest Pow API guide: https://github.com/pow-auth/pow_assent/blob/07a2c99377a71d413cb888158839aa6037eec9d0/guides/api.md

Let me know if it works for you.