-
Notifications
You must be signed in to change notification settings - Fork 82
Deployment fails if site password is set #111
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
Comments
Yes the site password is transmitted via a cookie. However there isn't currently any UI to set those cookies in ctfcli. For the time being I would disable the password and disable registration so no one can login to the authenticated parts. |
Yeah, I ended up disabling the password. I'm happy write a PR for this if you want? Maybe something like if config["config"]["site_password"]:
s.cookies["site_password"] = config["config"]["site_password"] Lines 82 to 83 in 45ffa11
|
Sure a PR wolud be great! However, I would prefer a more generic solution to the problem instead of specific keys. Perhaps if you could define an entire cookies sub-section or a cookies dictionary? |
Looks like ConfigParser doesn't parse |
ctf challenge sync
,install
anddeploy
fail if a site password is configured. I noticed asite_password
cookie is sent in the web UI though, and manual API calls succeed with it (egcurl
)The text was updated successfully, but these errors were encountered: