-
-
Notifications
You must be signed in to change notification settings - Fork 4
graylog_ldap_settings api call changed in version +4.x #8
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
We don't use Graylog 4.x ourselves (in part because of the auth system changes) so I'm not easily able to test against it; it might be necessary to make a second type/provider pair for the new auth system, since I'd want to preserve 3.x compatibility for the existing type. Maybe a |
think that will be indeed the way to go. especially when 3.x needs to be preserved. Is support for the enterprise features also needed ? In 4.x, the group mapping is only available in the enterprise version. And there are 4 types of backends (LDAP, Acitve-directory, okta and OIDC) Laast two are enterprise only. |
If you're willing to contribute a PR to add the I can see a couple of ways to go in terms of implementation:
Which method is more appropriate probably depends on what the API looks like on Graylog's side. |
see PR #9 |
Looking into it preparing PR to solve this.
This is now done using
api calls
The text was updated successfully, but these errors were encountered: