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

'Authentication Requests' graph should only include oxAuth authentication #1034

Closed
mzico opened this Issue Jul 9, 2018 · 5 comments

Comments

Projects
None yet
4 participants
@mzico
Contributor

mzico commented Jul 9, 2018

Seems like in clustered setup ( when there are multiple LDAP servers included and all of them are replicated ), 'Authentication Requests' graph is including some other 'authentication attempts' ( i.e. one ldap server is logging into another ldap server to mirror changed data ) along with user authentications.

handful_of_authentications

This graph should log only user authentication ( human ) attempts.

@mzico mzico added the enhancement label Jul 9, 2018

@mzico mzico added this to the 3.1.4 milestone Jul 9, 2018

@yurem

This comment has been minimized.

Contributor

yurem commented Jul 12, 2018

I suspect method which build graph not developed to show data mixed from few nodes.
In ou=metric tree we have sub-node ou=appliance_inum. We need to use it to build data per cluster node. And in the graph above we can add add either dropdown to select which node data to show or we can show all data on one graph. @mzico what do you think. Do you have idea?

@yurem

This comment has been minimized.

Contributor

yurem commented Aug 20, 2018

@mzico do you have testing data to reproduce this? Or how to reproduce it?

@yurem

This comment has been minimized.

Contributor

yurem commented Nov 29, 2018

@mzico Did you see same issue in 3.1.4?
In 3.1.4 we fixed issue which might led to this issue. In CE < 3.1.4 oxTrust and oxAuth both save statistic in oxAuth branch. In 3.1.4 we fixed this.

@mzico

This comment has been minimized.

Contributor

mzico commented Nov 29, 2018

@yurem I'll check with some customers who is using cluster with 3.1.4.

@yurem

This comment has been minimized.

Contributor

yurem commented Dec 5, 2018

@mzico Is it still work not well? we applied few fixes related to metrics in 3.1.4/3.1.5 Can you check it in 3.1.5 recent beta?

@yurem yurem closed this Dec 5, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment