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

[ZEPPELIN-3047] Let browser remember login and password #2679

Closed
wants to merge 1 commit into from

Conversation

mephi42
Copy link

@mephi42 mephi42 commented Nov 18, 2017

What is this PR for?

It makes Zeppelin login window compatible with Firefox password manager, as explained here: https://stackoverflow.com/questions/18586938/remember-password-with-angularjs-and-ng-submit

What type of PR is it?

Improvement

What is the Jira issue?

How should this be tested?

  • Open login window in Firefox, type anything. A suggestion to save username and password should appear.

Questions:

  • Does the licenses files need update? - No
  • Is there breaking changes for older versions? - No
  • Does this needs documentation? - No

@felixcheung
Copy link
Member

this looks reasonable but it is going to work the same in all other browsers?

@mephi42
Copy link
Author

mephi42 commented Nov 25, 2017

After some tinkering I managed to make it work in Firefox, Chrome and Safari on Mac.

@mephi42
Copy link
Author

mephi42 commented Nov 30, 2017

Improved the implementation: before it used form hiding for Chrome, and now it simply changes the action form attribute, which is not used anyway.

Also made sure that even though this change removes the code from the newly added "enter from username" change, the functionality still works: one can use enter from both username and password fields in order to submit the form.

@Leemoonsoo
Copy link
Member

Thanks @mephi42 for the improvement.

Here's message from CI

Looks like travis-ci is not configured for your fork.
Please setup by swich on 'zeppelin' repository at https://travis-ci.org/profile and travis-ci.
And then make sure 'Build branch updates' option is enabled in the settings https://travis-ci.org/mephi42/zeppelin/settings.

To trigger CI after setup, you will need ammend your last commit with
git commit --amend
git push your-remote HEAD --force

See http://zeppelin.apache.org/contribution/contributions.html#continuous-integration.

Could you try setup CI and see if everything goes green?

@mephi42
Copy link
Author

mephi42 commented Nov 30, 2017

Looks better this time, but one of sub-builds has timed out. I will push a dummy change to regtigger it.

@mephi42 mephi42 force-pushed the remember-password branch 2 times, most recently from 8170530 to ddf795d Compare December 1, 2017 09:06
@mephi42 mephi42 changed the title [ZEPPELIN-3047] Let Firefox remember login and password [ZEPPELIN-3047] Let browser remember login and password Feb 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants