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

Don't allow passwords containing line breaks #7670

Closed
2 tasks done
williamdes opened this issue Feb 24, 2024 · 4 comments
Closed
2 tasks done

Don't allow passwords containing line breaks #7670

williamdes opened this issue Feb 24, 2024 · 4 comments
Assignees
Labels
backlog Tasks and features handled by project maintainers priority: medium Medium priority issues that can wait, but should be addressed type: bug Something is causing incorrect behavior or errors

Comments

@williamdes
Copy link

Checklist

  • I have used the search function to see if someone else has already submitted the same bug report.
  • I will describe the problem with as much detail as possible.

App version

6.717

Where did you get the app from?

None

Android version

X

Device model

No response

Steps to reproduce

Start configuring your icloud.com email

Expected behavior

Password checks and it goes to next step

I did the setup on FairEmail at the same time and it worked.

Actual behavior

Server error: unmatch quote

Logs

02-24 22:58:27.993 20341 10889 V RealImapConnection: conn72600286 <<< #null# [OK, [CAPABILITY, XAPPLEPUSHSERVICE, IMAP4, IMAP4rev1, SASL-IR, AUTH=ATOKEN, AUTH=PLAIN], (2409B40-f42d5d1bd981) mr36p00im-tygg11200301.me.com]
02-24 22:58:27.994 20341 10889 D RealImapConnection: Saving [IMAP4, AUTH=PLAIN, XAPPLEPUSHSERVICE, SASL-IR, AUTH=ATOKEN, IMAP4REV1] capabilities for conn72600286
02-24 22:58:27.995 20341 10889 V RealImapConnection: conn72600286>>> 1 AUTHENTICATE PLAIN 

02-24 22:58:28.166 20341 20341 D InsetsController: show(ime(), fromIme=true)
02-24 22:58:28.223 20341 10889 V RealImapConnection: conn72600286<<<#+# []
02-24 22:58:28.651 20341 10889 V ImapResponseParser: conn72600286<<<#1# [NO, [AUTHENTICATIONFAILED], Authentication Failed]


RealImapConnection: conn72600286>>> 2 LOGIN "myusername" "ykkl-xxx-xxxt-kkki
02-24 22:58:28.655 20341 10889 V RealImapConnection: 
02-24 22:58:28.655 20341 10889 V RealImapConnection: "
02-24 22:58:28.848 20341 10889 V ImapResponseParser: conn72600286<<<#2# [BAD, unmatch quote]
@williamdes williamdes added type: bug Something is causing incorrect behavior or errors unconfirmed Newly reported issues awaiting triage or confirmation labels Feb 24, 2024
@williamdes
Copy link
Author

There is some code and tests on GitHub

https://github.com/Kopano-dev/vmime/blob/431305e11d847d93b6d67da8867d272d872c127a/tests/net/imap/IMAPParserTest.cpp#L68

AUTH=ATOKEN maybe means apple token. I have no idea. Seems like they have a difference in the protocol implementation

@cketti
Copy link
Member

cketti commented Feb 24, 2024

Are you using a password manager or some other tool that somehow managed to enter a password containing two line breaks at the end?

@williamdes
Copy link
Author

Hmm, indeed maybe my copy paste from their website did this

@williamdes
Copy link
Author

Yeah that was it !
But the password input does not reflect such characters, FairEmail input does. That's how I did see that I needed to use "paste in raw format"

@cketti cketti changed the title Unable to configure icloud.com email Don't allow passwords containing line breaks Feb 24, 2024
@cketti cketti removed the unconfirmed Newly reported issues awaiting triage or confirmation label Feb 24, 2024
@wmontwe wmontwe added priority: medium Medium priority issues that can wait, but should be addressed backlog Tasks and features handled by project maintainers labels Feb 29, 2024
@wmontwe wmontwe self-assigned this Feb 29, 2024
@cketti cketti closed this as completed Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Tasks and features handled by project maintainers priority: medium Medium priority issues that can wait, but should be addressed type: bug Something is causing incorrect behavior or errors
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants