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

Sharing with password -- Pressing enter to save password cannot be discovered #57

Closed
fmms opened this issue Oct 23, 2012 · 5 comments
Closed

Comments

@fmms
Copy link
Contributor

fmms commented Oct 23, 2012

I tried to reproduce http://article.gmane.org/gmane.comp.kde.devel.owncloud/6035 to open a bug for it.

However, I came to realize that it does not work at all. Setting the expiration date or enabling sharing is saved without an issue.

However, if i open the share menu and enter a password (press the lock icon) this is not saved. Looking at Firebug, nothing is transfered to the server doing so.

However there is no error message and thus I by accident shared without any password at all.

This is FF16 and ownCloud git.

@LukasReschke
Copy link
Member

This may be related to oc-1949:

You have to hit enter after you type in a password and then it should say 'Password protected' inside the textbox. I'm not sure if this is the best user experience.

Can you confirm this?

@fmms
Copy link
Contributor Author

fmms commented Oct 23, 2012

Thanks alot @LukasReschke this was indeed the issue.

I think this is totally none discoverable.

Submitting the password after every key stroke seems like a security issue, as an attacker could see how long it is by counting requests.
I would suggest submit it on leaving the field or closing the menu.

Thanks for you quick help, would have never discovered that.

@ghost ghost assigned MTGap Oct 23, 2012
@LukasReschke
Copy link
Member

I would suggest submit it on leaving the field or closing the menu.

This sounds resonable.

@MTGap what do you think?

@jancborchardt
Copy link
Member

Yes, submit on blur is good, and there needs to be some feedback like on the email field in the personal settings. (Or better yet, like the account settings fields here on Github with just a checkmark.)

@schiessle
Copy link
Contributor

I changed it to set the new password as soon as the password filed loses focus.

aditmeno pushed a commit to aditmeno/Aeolus that referenced this issue Apr 2, 2015
removing php-cloudfiles, replaced by php-opencloud
@lock lock bot locked as resolved and limited conversation to collaborators Aug 25, 2019
@lock lock bot unassigned MTGap Aug 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants