Skip to content
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

Monzo now requires redirect_uri #12

Closed
Sheaffy opened this issue Jan 19, 2018 · 5 comments
Closed

Monzo now requires redirect_uri #12

Sheaffy opened this issue Jan 19, 2018 · 5 comments
Assignees
Labels
question Further information is requested

Comments

@Sheaffy
Copy link
Contributor

Sheaffy commented Jan 19, 2018

As per Monzo docs https://monzo.com/docs/#acquire-an-access-token

They now require a redirect URI,

pip installed version error code
oauthlib.oauth2.rfc6749.errors.InvalidClientIdError: (invalid_request) The supplied redirect_uri is different from the one used to issue this code

@Sheaffy Sheaffy changed the title Monzo now requires request_uri Monzo now requires redirect_uri Jan 19, 2018
@Sheaffy
Copy link
Contributor Author

Sheaffy commented Jan 19, 2018

My bad, didn't see config.

@Sheaffy Sheaffy closed this as completed Jan 19, 2018
@pawelad
Copy link
Owner

pawelad commented Jan 19, 2018

Wait, I'm a little bit lost, so did it work? : -)

When did you see that error? On pip install pymonzo?

@Sheaffy
Copy link
Contributor Author

Sheaffy commented Jan 19, 2018

I did get it to work yes, Basically you can't have a custom redirect uri if you install via pip.. you need to edit the config.py to put in your own redirection uri.. If they don't match you get the above error.

@pawelad
Copy link
Owner

pawelad commented Jan 19, 2018

Hm. I'd be OK with making it configurable via an environment variable, which I think makes sense. I'll go ahead and open an issue for that.

@pawelad pawelad added the invalid This doesn't seem right label Jan 19, 2018
@pawelad pawelad added question Further information is requested and removed invalid This doesn't seem right labels Jan 26, 2018
@pawelad
Copy link
Owner

pawelad commented Feb 16, 2018

OK, so this is now possible in v0.11.0 : )

@pawelad pawelad self-assigned this Feb 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants