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

Login errors #439

Closed
strowi opened this issue Feb 26, 2021 · 3 comments · Fixed by #440
Closed

Login errors #439

strowi opened this issue Feb 26, 2021 · 3 comments · Fixed by #440

Comments

@strowi
Copy link

strowi commented Feb 26, 2021

Hi,

thx for the ttrss-app ;)

I'm having trouble syncing the feeds with the app, error message says login is incorrect.

Debugging i found some changes in ttrss-repo and could pinpoint problems started with the following commit:
https://git.tt-rss.org/fox/tt-rss/commit/59c14e9c0001bc7a01763ecc7d3042dcde978a1a

~> curl -d '{"op": "login", "user": "xyz", "password": "123"}' https://rss.abc.net/api/ -vL

still works with current version though.

regards,
Roman

@Obererpel
Copy link

Yep, ran into this problem, too. The app converts the password to base64, but ttrss does not accept this anymore.

This is a game breaker, so someone should fix this asap. Is Nils still actively developing this app or should I learn how to compile android apps now :)

@webworxshop
Copy link
Contributor

I've just submitted a trivial PR fixing this issue, which works well for me.

For those that use this app on a daily basis and can't wait for this to be merged/released, I've uploaded a patched APK at: https://cloud.webworxshop.com/s/qsS57FRLwLkJAZX (usual caveats about installing random APKs from the Internet apply).

@nilsbraden
Copy link
Owner

Thanks for providing the fix! I didn't update the server recently and didn't notice this yet.

nilsbraden added a commit that referenced this issue Mar 14, 2021
 * Fix #445: Close app with "Go back after mark all read" on main page
 * Fix #439: Login issues due to removed base64 encoding of credentials
 * Fix #441: Mark as read generates a warning on the server
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

Successfully merging a pull request may close this issue.

4 participants