[GH-ISSUE #195] Normal functioning? #160

New issue

Closed

opened 2026-02-25 21:34:19 +03:00 by kerem · 2 comments

kerem commented

2026-02-25 21:34:19 +03:00

Owner

Originally created by @mwolfeu on GitHub (May 14, 2017).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/195

Originally assigned to: @jasonmunro on GitHub.

I have:
auth_type=dynamic
but accidentally left in:
; modules[]=dynamic_login

Regardless, I can type in a new user name/password in a browser and save it in a text file. Is this intended?

Originally created by @mwolfeu on GitHub (May 14, 2017). Original GitHub issue: https://github.com/cypht-org/cypht/issues/195 Originally assigned to: @jasonmunro on GitHub. I have: auth_type=dynamic but accidentally left in: ; modules[]=dynamic_login Regardless, I can type in a new user name/password in a browser and save it in a text file. Is this intended?

kerem

2026-02-25 21:34:19 +03:00

closed this issue
added the
bug

framework
labels

kerem commented

2026-02-25 21:34:19 +03:00

Author

Owner

@jasonmunro commented on GitHub (May 15, 2017):

If you set the auth_type to dynamic, and you don't activate the module, it uses the "Auth type none" class (which really only exists for unit tests), that returns true for any username/password combination. This is a bug IMO. We should just "die" like we do for other misconfigured authentication configurations. Thanks for this feedback!

@jasonmunro commented on GitHub (May 15, 2017): If you set the auth_type to dynamic, and you don't activate the module, it uses the "Auth type none" class (which really only exists for unit tests), that returns true for any username/password combination. This is a bug IMO. We should just "die" like we do for other misconfigured authentication configurations. Thanks for this feedback!

kerem commented

2026-02-25 21:34:19 +03:00

Author

Owner

@jasonmunro commented on GitHub (May 15, 2017):

Fixed in github.com/jasonmunro/cypht@40ca661d43 thanks again!

@jasonmunro commented on GitHub (May 15, 2017): Fixed in https://github.com/jasonmunro/cypht/commit/40ca661d436ded01c4731f13220cab13d95c5467 thanks again!