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

Update user store REST API to test connection for both JDBC and LDAP types #392

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

isuruhettiarachchi
Copy link

Purpose

  • Update test-connection user store REST API to support LDAP user store types.

Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Updated the yaml to accept a general request object for both JDBC and LDAP user store types. Updated the model keeping the previous variables to maintain backward compatibility.
Now test-connection API accepts a request body as below as well in addition to the previous request body.

{
    "typeId": "VW5pcXVlSURSZWFkT25seUxEQVBVc2VyU3RvcmVNYW5hZ2Vy",
    "properties": [
        {
            "name": "ConnectionURL",
            "value": "ldap://localhost:10390"
        },
        {
            "name": "ConnectionName",
            "value": "uid=admin,ou=system"
        },
        {
            "name": "ConnectionPassword",
            "value": "admin"
        }
    ]
}

Related PRs

Depends on the wso2/carbon-identity-framework#4230
Fixes wso2/product-is#1528

@isuruhettiarachchi isuruhettiarachchi marked this pull request as draft October 3, 2022 07:25
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

Successfully merging this pull request may close these issues.

Provide an option to test the connectivity when adding a user store
1 participant