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

statusCheckInterval is too low #48

Open
alexandre-zia-ifood opened this issue Jun 14, 2020 · 1 comment
Open

statusCheckInterval is too low #48

alexandre-zia-ifood opened this issue Jun 14, 2020 · 1 comment

Comments

@alexandre-zia-ifood
Copy link

Hi,

I was wondering, why is statusCheckInterval too low?
The reason whay someone would use gluu-radius is to use twosetp authentication and take advantage of 2FA, this implies waiting for a human to take some action (approve or not on Supergluu) and no human could be that fast, and it doesn't matter if it took 1 second or two to detect step one auth success.
Shouldn't this be set to 1000 or 2000 ?

private static final long statusCheckInterval = 50; // in ms

@uprightech
Copy link
Collaborator

Hello Alexandre ,
The statusCheckInterval is the interval between which the radius server will poll the Gluu server about 2FA authentication for the user. Too fast (too low a value) and you may poll the way too much , possibly impacting performance , too slow (higher value) and there will be a significant delay between when the user hits Approve on his phone and him getting authenticated.

That said , there is another value called the authenticationTimeout which is stored in Gluu server itself , and which you can set from the UI. That value is the value I think you're looking for.
Notice how from this loop it is used , and how the statusCheckInterval is used to "slow things down".

Hope the explanation helps.

It may be useful to know which specific issue you're having , if any,

Best Regards,
Rolain

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