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

Field 'enabled_clients' is not included in the Connection definition #945

Open
5 tasks done
jkillian-psc opened this issue Sep 27, 2023 · 6 comments
Open
5 tasks done
Labels
bug This points to a verified bug in the code

Comments

@jkillian-psc
Copy link

Checklist

  • I have looked into the Readme, Examples, and FAQ and have not found a suitable solution or answer.
  • I have looked into the API documentation and have not found a suitable solution or answer.
  • I have searched the issues and have not found a suitable solution or answer.
  • I have searched the Auth0 Community forums and have not found a suitable solution or answer.
  • I agree to the terms within the Auth0 Code of Conduct.

Description

Connection interface is missing enabled_clients field that is included in the response. The Management client API documentation for the operation (https://auth0.com/docs/api/management/v2/connections/get-connections-by-id) also makes no mention of the field either.

image

Reproduction

  1. Initialize the Auth0 Management API client using library
  2. Call the get( ) method on the ConnectionsManager
  3. Print the result returned from the method

Additional context

No response

node-auth0 version

4.0.1

Node.js version

18.16.1

@jkillian-psc jkillian-psc added the bug This points to a verified bug in the code label Sep 27, 2023
@frederikprijck
Copy link
Member

Thanks for adding an issue, I am looking into this and will report back when I have more info. For the time being, you might want to unblock yourself typewise with some unfortunate casting or manual typing.

@CooperBills
Copy link

FWIW, we're also updating our client from an older version to 4.0.1 and we noticed this field is missing from the newer typings as well.

I see it's still in the actual object though, so we'll unblock with some manual typing & sanity checks, but I'm chiming in here to note @jkillian-psc isn't the only one seeing this.

@frederikprijck
Copy link
Member

Thanks for that @CooperBills , yeah i have confirmed this with other teams and working on getting this fixed.

@Gentracker1
Copy link

Hi,
We are on 4.3.1 and we used to update the connection. Is there any other way to check the enabled clients?
Our code is in production and we want to upgrade to the latest version.
Thanks

@tgillitzer
Copy link

@frederikprijck Any timeline on getting this fixed? It makes it pretty hard to update a connection's enabled clients if you can't get the ones that are already enabled.

@shawnb-gv
Copy link

The only temp workaround I've found is to do something like connections["enabled_clients"] rather than connections.enabled_clients which throws the type error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This points to a verified bug in the code
Projects
None yet
Development

No branches or pull requests

6 participants