[GitHub] accumulo pull request #287: ACCUMULO-4688 Turn off autocomplete for the Shel...

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[GitHub] accumulo pull request #287: ACCUMULO-4688 Turn off autocomplete for the Shel...

ctubbsii
GitHub user joshelser opened a pull request:

    https://github.com/apache/accumulo/pull/287

    ACCUMULO-4688 Turn off autocomplete for the ShellServlet's form passw…

    …ord field

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/joshelser/accumulo 4688-autocomplete-off

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/accumulo/pull/287.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #287
   
----
commit ef52d6c59c4ea65b1807a5da5164c8da8cda3886
Author: Josh Elser <[hidden email]>
Date:   2017-07-31T21:25:26Z

    ACCUMULO-4688 Turn off autocomplete for the ShellServlet's form password field

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[GitHub] accumulo issue #287: ACCUMULO-4688 Turn off autocomplete for the ShellServle...

ctubbsii
Github user ctubbsii commented on the issue:

    https://github.com/apache/accumulo/pull/287
 
    Why impose this inconvenience on users? Shouldn't users decide which client-side features to use themselves? Also, this servlet is removed in the master branch. This would only negatively affect users upgrading to a bugfix version. I don't think we should do this.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[GitHub] accumulo issue #287: ACCUMULO-4688 Turn off autocomplete for the ShellServle...

ctubbsii
In reply to this post by ctubbsii
Github user ctubbsii commented on the issue:

    https://github.com/apache/accumulo/pull/287
 
    This also undermines user's use of password managers for improved security. Saved passwords in secure password managers is a good thing for web security. We don't want to discourage those users.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[GitHub] accumulo pull request #287: ACCUMULO-4688 Turn off autocomplete for the Shel...

ctubbsii
In reply to this post by ctubbsii
Github user joshelser closed the pull request at:

    https://github.com/apache/accumulo/pull/287


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Loading...