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

ISO 8859-1 special and accented characters not supported #1871

Open
2 tasks done
ilike2burnthing opened this issue Sep 15, 2023 · 4 comments
Open
2 tasks done

ISO 8859-1 special and accented characters not supported #1871

ilike2burnthing opened this issue Sep 15, 2023 · 4 comments
Labels
Status: Needs Triage New Issue needing triage Type: Bug Issue is a bug

Comments

@ilike2burnthing
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing open and closed issues

Current Behavior

For an indexer using ISO 8859-1 encoding, logins fail when using a password which contains special or accented characters, see rows Ax to Fx here - https://en.wikipedia.org/wiki/ISO/IEC_8859-1#Code_page_layout

Expected Behavior

Works, as in Jackett.

Steps To Reproduce

  1. have password containing special or accented character for ISO 8859-1 encoded tracker (e.g. Deildu, Genesis-Movement, TribalMixes)
  2. try to add indexer
  3. fail with 'wrong credentials' error, despite them being correct

Environment

- OS: Windows 10 Sandbox
- Prowlarr: 1.8.6.3946
- Docker Install: No
- Using Reverse Proxy: No
- Browser: MS Edge

What branch are you running?

Master

Trace Logs?

N/A - just normal failed login logs

Trace Logs have been provided as applicable. Reports may be closed if the required logs are not provided.

  • I have read and followed the steps in the wiki link above and provided the required trace logs - the logs contain trace - that are relevant and show this issue.
@ilike2burnthing ilike2burnthing added Status: Needs Triage New Issue needing triage Type: Bug Issue is a bug labels Sep 15, 2023
@mynameisbogdan
Copy link
Contributor

Jackett seems to use a mix of UTF-8 and Latin1 on this one.

@mynameisbogdan
Copy link
Contributor

Tried with TribalMixes earlier, but after I added è to my password I get the following message and can't login anymore even with the old password. The only way is to reset the password, so on this I'm inclined to say it's rather a site issue.

Update failed!

That doesn't look like a valid email address.

@ilike2burnthing
Copy link
Contributor Author

Odd, I've just tested Deildu, Genesis-Movement, and TribalMixes with è in their passwords, and they all worked in browser and Jackett. That's also a weird error to get for a password issue. I can cause that error by add a trailing/leading space in the email field, or using a + in the email address.

@mynameisbogdan
Copy link
Contributor

I'll test the others when I'll have more time available but for Tribal I can't even change the password. Only the one from the reset works for me. 😆

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Triage New Issue needing triage Type: Bug Issue is a bug
Projects
None yet
Development

No branches or pull requests

2 participants