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.

Sign up for GitHub

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

Jump to bottom

Auto-Config Case Sensitivity #1005

Closed
Seandals opened this issue Sep 14, 2020 · 4 comments · Fixed by #1009
Closed

Auto-Config Case Sensitivity #1005

Seandals opened this issue Sep 14, 2020 · 4 comments · Fixed by #1009
Assignees
@Expl0dingBanana
Labels
More Info Requires more info investigate the issue

Comments

@Seandals
Copy link
Contributor

Hey there. While flashing my boxes tonight to x64 I ran into some issues due to the MAD Backend username login in PogoDroid being case sensitive but in the Auto-Config PogoDroid Configuration options it defaults all inputs to lowercase, causing logins to fail. Can this field be changed to respect character case as entered?
@Expl0dingBanana Expl0dingBanana added the More Info Requires more info investigate the issue label Sep 17, 2020
@Expl0dingBanana
Copy link
Collaborator

This was fixed in d85881d. What commit hash are you currently on?

@Seandals
Copy link
Contributor Author

Seandals commented Sep 18, 2020 •
edited
Loading

This was fixed in d85881d. What commit hash are you currently on?

I pull every night at midnight. Still forces to lowercase on current master.

@krzee
Copy link
Collaborator

krzee commented Sep 18, 2020

banana, it was google that needed to be all lowercase... i didnt know the MAD backend is case sensitive for emails... but if it is then then it needs to be handled differently from google email

@krzee
Copy link
Collaborator

krzee commented Sep 18, 2020 •
edited
Loading

or maybe the right solution is to make the backend not be case sensitive for emails? as email addresses should not be case sensitive in the first place

@Expl0dingBanana Expl0dingBanana mentioned this issue Sep 19, 2020
Merged
@Expl0dingBanana Expl0dingBanana linked a pull request Sep 19, 2020 that will close this issue
Fix PD Case Sensitivity #1009
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Expl0dingBanana Expl0dingBanana

Labels
More Info Requires more info investigate the issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix PD Case Sensitivity Map-A-Droid/MAD
3 participants
@krzee @Expl0dingBanana @Seandals