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

sx_authenticate redirects to a "not valid uri" #349

Open
RockyRoad29 opened this issue Feb 13, 2018 · 6 comments
Open

sx_authenticate redirects to a "not valid uri" #349

RockyRoad29 opened this issue Feb 13, 2018 · 6 comments

Comments

@RockyRoad29
Copy link
Contributor

I was not able to get an OAuth token
M-x sx-authenticate sends me to uri (I obliterated here the client-id) https://stackoverflow.com/oauth/dialog?client_id=####&redirect_uri=http%253A%252F%252Fseanallred.com%252Fsx.el%252Fauth%252Fauth.htm&scope=read_inbox%2cno_expiry%2cprivate_info%2cwrite_access.
The server answers:
http%3A%2F%2Fseanallred.com%2Fsx.el%2Fauth%2Fauth.htm is not a valid uri

I think the redirect_uri looks like encoded twice.

@RockyRoad29 RockyRoad29 changed the title sx_authenticate redirects to a "not valid uri" for SE API sx_authenticate redirects to a "not valid uri" Feb 13, 2018
@RockyRoad29
Copy link
Contributor Author

RockyRoad29 commented Feb 13, 2018

This PR #350 fixed it for me. But for some reason CI fails. I leave it to you from here.

@basil-conto
Copy link
Contributor

basil-conto commented Feb 27, 2018

Aren't this issue and the related PR #350 duplicates of #342?

vermiculus added a commit that referenced this issue Apr 25, 2018
@unhammer
Copy link

unhammer commented Jan 25, 2019

in melpa-stable 0.4 I got Could not parsed uri [http%3A%2F%2Fseanallred.com%2Fsx.el%2Fauth%2Fauth.htm].

I upgraded to (unstable) melpa 20190114.1523 and it seems to be working there.

(edited for clarity)

@vermiculus
Copy link
Owner

I've not yet made a release (and I don't plan to for some time as time is scarce). Are you able to apply any necessary patches locally or (probably better) switch to using MELPA or some system that pulls HEAD?

@basil-conto
Copy link
Contributor

@RockyRoad29 Can you still reproduce this issue?

@matthew-piziak
Copy link

@vermiculus Fixed for me now! Thank you!

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

No branches or pull requests

5 participants