The Wayback Machine - http://web.archive.org/web/20201203073353/https://github.com/nextcloud/server/issues/24141
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

[sharebymail] Before password validation, clean pasted password with whitespace before/after and \ #24141

Open
medcloud opened this issue Nov 14, 2020 · 2 comments

Comments

@medcloud
Copy link
Contributor

@medcloud medcloud commented Nov 14, 2020

How to use GitHub

  • Please use the 👍 reaction to show that you are interested into the same feature.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Is your feature request related to a problem? Please describe.
Share by mail let user to share document with foreigner to get or send sensitive document protected by a password

Most of users copy/past password from the email they have received as it is faster and more convenient with automatic generated complex passwords.

However, the copy of selected text (password) leads most of the time to copy the invisible \n before or after, or a whitespace.

Describe the solution you'd like
The solution that could be applied is to erase \n and whitespace before/after of pasted text (password) just before to call the service.

Describe alternatives you've considered
No alternatives except to select again and again the text : most of people not succeding call back the nextcloud account owner for a new link thinking that the password in invalid.

Additional context
If the sender is familiar with nextcloud, most of the time the recipient (sometimes old peaple) are not at ease with computers

@stavoxnetworks
Copy link

@stavoxnetworks stavoxnetworks commented Nov 16, 2020

I can take this if you like. I would be glad to join the project.

@skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Nov 25, 2020

@stavoxnetworks please do :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.