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

Disable password prompt #132

Closed
johanricher opened this issue Dec 17, 2018 · 2 comments
Closed

Disable password prompt #132

johanricher opened this issue Dec 17, 2018 · 2 comments
Milestone

Comments

@johanricher
Copy link

Since a lot of use cases for static-GTFS-manager are covered by a local installation on a desktop computer, wouldn't it be relevant to allow those users to disable the password locking? Would it be possible to easily change this setting in a config file?

Even for a server installation, I think it would be preferable to have a single password prompt at the start of a session. The current implementation forces users to re-enter the password at every change on every page, which is quite annoying in my opinion.

@answerquest
Copy link
Collaborator

answerquest commented Dec 18, 2018

Hi @johanricher , you're right. I had put in the password feature initially when this program was meant to run on a web server and there was a big risk of others on the network accidentally messing with things. I haven't used any frameworks that natively enable remembering a user/browser session, and I wanted to avoid using any cookies, so this was a workaround.

But I see your point that for users that are actually using the tool fully (ie, making many edits), this is a major hindrance.

I'll look into various options to make things easier. Putting something in the config folder may be the best path to take. There are users who are deploying this on a public-facing web server, so I don't want to completely remove the editing protections.

For now, here's a workaround at the Javascript end; it'll work without having to regenerate the binaries:

  1. Open js/commonfuncs.js
  2. Find this code block around line 90:
});
// ############################
// FUNCTIONS
  1. Insert this line just above those lines:
document.getElementById("password").value = "program";
  1. Change the "program" value above to your password if you have set it differently.

  2. There, now the password field should get auto-loaded at all pages.

@johanricher
Copy link
Author

Thanks for your answer @answerquest (and all your good work on this awesome project). Looking forward to an easy configuration of this. I'll try the workaround in the meantime.

@answerquest answerquest added this to the v3.4.0 milestone Dec 19, 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
Development

No branches or pull requests

2 participants