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

Issue with "FortiAuthenticator" in @site/docs/reference/integrations/forti-authenticator.md #743

Closed
fahad0707 opened this issue Aug 4, 2021 · 2 comments · Fixed by demisto/content#14430
Assignees

Comments

@fahad0707
Copy link

Looks you are referencing "localusers" and the integration is failing for LDAP users.
I replaced /localusers with /ldapusers and integration worked.
I am a bit skeptical about the change I made as I have no or very little programming skills.
However I think you might have understood and may be publish the new content package to marketplace.


    res = requests.patch(BASE_URL + 'localusers/' + userID + '/', data=jsonData, auth=(USER_NAME, PASSWORD), verify=USE_SSL)

    if res.status_code == 202:
        user_context.append({
            'username': userItems["objects"][0]["username"],
            'email': userItems["objects"][0]["email"],
            'active': active,
            'id': userItems["objects"][0]["id"]
        })
@welcome
Copy link

welcome bot commented Aug 4, 2021

Hi and welcome to our Content Developer Docs project!
Thanks and congrats for opening your very first issue.
If you know the fix, you may also submit a PR to fix the issue if you like!

@jasonlo82
Copy link

Looks you are referencing "localusers" and the integration is failing for LDAP users.
I replaced /localusers with /ldapusers and integration worked.
I am a bit skeptical about the change I made as I have no or very little programming skills.
However I think you might have understood and may be publish the new content package to marketplace.

    res = requests.patch(BASE_URL + 'localusers/' + userID + '/', data=jsonData, auth=(USER_NAME, PASSWORD), verify=USE_SSL)

    if res.status_code == 202:
        user_context.append({
            'username': userItems["objects"][0]["username"],
            'email': userItems["objects"][0]["email"],
            'active': active,
            'id': userItems["objects"][0]["id"]
        })

@fahad0707 Thanks for reporting this. The "localusers" should be replaced by the variable userType. I will update it ASAP.

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