Skip to content
This repository has been archived by the owner on Oct 10, 2024. It is now read-only.

mysql server has gone away --> Token Validation succeeds #1

Open
martinhaase opened this issue Apr 25, 2019 · 1 comment
Open

mysql server has gone away --> Token Validation succeeds #1

martinhaase opened this issue Apr 25, 2019 · 1 comment

Comments

@martinhaase
Copy link

martinhaase commented Apr 25, 2019

Hi Jordan,
the TokenValidator is calling getTokenList, which succeeds also if the PI server has no connection to its database, as piConnection:getTokenList does not call checkAPISuccess. This means, in case of an unstable database, that 2FA is bypassed. We see this as an security issue, the plugin should not let the flow succeed in this case.
Regards,
Martin

@wraezor
Copy link
Owner

wraezor commented Jul 25, 2019

This is a design choice. Currently this implementation is designed to fail open in the case you identified (missing database server) but also in other cases, such as Shibboleth being unable to securely contact the privacyIDEA server (due to bad certificate, etc).

I don't have any plans presently to fix this, but I am open to a pull request if someone else would like to try.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants