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

Support include_credential param for ListIdentities API #3190

Closed
3 of 6 tasks
bcordobaq opened this issue Mar 27, 2023 · 3 comments
Closed
3 of 6 tasks

Support include_credential param for ListIdentities API #3190

bcordobaq opened this issue Mar 27, 2023 · 3 comments
Labels
feat New feature or request.

Comments

@bcordobaq
Copy link

Preflight checklist

Describe your problem

Currently, the List Identity API service returns the Identity data without the credentials section, and I you want to get it, you need to use the individual Get Identity service.

Describe your ideal solution

Include the credentials section as optional only if the user adds the include_credential param when calling the ListIdentities API. It's the same param that already has the Get Identity API.

Workarounds or alternatives

Return by default the credentials section, but it wouldn't be a good solution, since it's not solved like this for the other related Identity APIs.

Version

0.11

Additional Context

No response

@bcordobaq bcordobaq added the feat New feature or request. label Mar 27, 2023
@LukaJaj
Copy link

LukaJaj commented Dec 2, 2023

#3652

@borisroman
Copy link
Contributor

I think this can be closed once #3343 is merged.

@jonas-jonas
Copy link
Member

Fixed via #3343

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants