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

Closed
5 tasks done
jkillian-psc opened this issue Sep 27, 2023 · 9 comments
Closed
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.

@icco
Copy link

icco commented Jun 6, 2024

I have created #1009 to fix this, as I filed a support ticket and they said they were not planning on prioritizing a fix.

@frederikprijck
Copy link
Member

frederikprijck commented Jun 7, 2024

@icco We provided the support team with a (very ugly) workaround to unblock you immediately, while we acknowledged the need to have this fixed and the team is looking in fixing this properly so you no longer need such workaround, but because there is a (very ugly) workaround, a few other things have higher priority. But that does not mean this isn't going to be looked at.

Apologies this issue went stale tho, it kinda got lost from our radar, but you did help bring it back to our attention, thank you for that.

@gyaneshgouraw-okta
Copy link
Contributor

Fix for this issue is released in https://www.npmjs.com/package/auth0/v/4.6.0

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

8 participants