grp / newsyc

An iPhone Hacker News client.
http://newsyc.me/
BSD 2-Clause "Simplified" License
1.11k stars 280 forks source link

Instapaper not working #109

Open guiambros opened 11 years ago

guiambros commented 11 years ago

Since last month, saving articles to Instapaper suddenly stopped working. The exact same user/pass that works fine via a browser, receives an "Unable to Authenticate. Make sure your username and password are correct" error message.

I tested in 3 different iOS devices (one iPhone 5 with 6.1, another with 6.1.4, and an iPad Mini with 6.1). Also tried using different Instapaper accounts and even created a fresh one just to test. Same error message in all cases.

I sniffed the connection with Charles Proxy [1], and I can see the connection being made to GET /api/authenticate, but strangely there is no username or password being appended to the query string.

I looked at the source code, more specifically InstapaperAuthenticator.m, beginAuthentication(), and seems fine to me. Besides, the code was last touched more than 4 months ago, and for sure it worked for me after this.

The only thing I can think of is some change in Instapaper's side, but this wouldn't explain why user/pass is not being passed in the query string. I already ruled out the OAUTH, as the Simple API method (which is used by newsyc) is still working properly.

Is Instapaper still working for any of you? Any ideas how to troubleshoot?

Thanks a lot.

debug newsyc

newsyc commented 11 years ago

I'll take a look this weekend.

Sent from my iPhone

On Jun 29, 2013, at 10:56 AM, guiambros notifications@github.com wrote:

Since last month, saving articles to Instapaper suddenly stopped working. The exact same user/pass that works fine via a browser, receives an "Unable to Authenticate. Make sure your username and password are correct" error message.

I tested in 3 different iOS devices (one iPhone 5 with 6.1, another with 6.1.4, and an iPad Mini with 6.1). Also tried using different Instapaper accounts and even created a fresh one just to test. Same error message in all cases.

I sniffed the connection with Charles Proxy [1], and I can see the connection being made to GET /api/authenticate, but strangely there is no username or password being appended to the query string.

I looked at the source code, more specifically InstapaperAuthenticator.m, beginAuthentication(), and seems fine to me. Besides, the code was last touched more than 4 months ago, and for sure it worked for me after this.

The only thing I can think of is some change in Instapaper's side, but this wouldn't explain why user/pass is not being passed in the query string. I already ruled out the OAUTH, as the Simple API method (which is used by newsyc) is still working properly.

Is Instapaper still working for any of you? Any ideas how to troubleshoot?

Thanks a lot.

[image: debug newsyc]https://f.cloud.github.com/assets/205000/726273/72093a0a-e0e4-11e2-9b7c-c42587875038.png

— Reply to this email directly or view it on GitHubhttps://github.com/Xuzz/newsyc/issues/109 .

shedd commented 11 years ago

I'm also running into this issue on my iPhone 4 (iOS 6.1.3) and iPad 3 (iOS 6.1.3).

I get an "Error saving" error when trying to save to Instapaper.

I went into Settings to force the Instapaper logout and tried to log in again, and get the authentication error as described. However, the username/password works on the web site. I also uninstalled and reinstalled the app to no avail.

grp commented 11 years ago

Yes, this appears to be an issue on Instapaper's end. I haven't received any response from them so far.

shedd commented 11 years ago

Still nothing from Instapaper on this?

amitm commented 11 years ago

Hey Guys,

I handle instapaper's backend right now and I took a look. It doesn't look like the client is posting any sort of login information to the authenticate method.

grp commented 11 years ago

I got a pull request recently to fix it; I was hitting the API without a "www." before instapaper.com. I haven't pushed the update to the App Store though.

shedd commented 11 years ago

Thanks so much for fixing this. When might the App Store update get pushed?

grp commented 11 years ago

Once I have some time to add iOS 7 support, probably.

shedd commented 10 years ago

This issue appears to be fixed now! Thanks!