Skip to content
This repository has been archived by the owner on Jul 24, 2018. It is now read-only.

OC 6.0.0 When sharing WebODF documents by link and e-mail with password: password input is visible (DON'T DO THAT) #165

Closed
Wikinaut opened this issue Dec 12, 2013 · 6 comments

Comments

@Wikinaut
Copy link
Contributor

(originally filed as owncloud/core#6368, now closed there.)

OC 6.0.0 When sharing WebODF documents by link and e-mail with password: password input is visible (DON'T DO THAT)

No milestone
No one is assigned

When you as user Alice share - this is one of the new features of OC 6 - an ODT document with Bob and send a mail with a hash link and password to Bob,

Bob visits that link and is prompted to enter the password.

Bug:

The input of the password is not hidden it is visible in clear text, which can be dangerous in public environments. Please fix this, it is probably only a matter of adding type="password" to the input tag.

Pull request coming soon.

@Wikinaut
Copy link
Contributor Author

fixed in 0d9a73d

@menelic
Copy link

menelic commented Dec 13, 2013

When will this be included in the documents app?And how can I get a secure but stable pre-release version in case this takes longer? Sorry for the end-user question but I am using the app for collaboration so I am concerned about the security implications of this. Please let me know, thanks!

@Wikinaut
Copy link
Contributor Author

@menelic hi, until this tiny fix 0d9a73d is actually backported (or ported) to the distribution files, you can easily(!) apply this change manually, the full filename is

/your-server/owncloud/apps/documents/template/public.php

change

- <input type="text" name="password" placeholder="<?php p($l->t('Password')) ?>" />
+ <input type="password" name="password" placeholder="<?php p($l->t('Password')) ?>" />

@Wikinaut
Copy link
Contributor Author

I propose, that this fix is backported to 6.0.0

@Wikinaut Wikinaut reopened this Dec 13, 2013
karlitschek pushed a commit that referenced this issue Dec 13, 2013
@karlitschek
Copy link

backported a22b358

@karlitschek
Copy link

backported

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

3 participants