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

Add deprecation note for DELETE /users (all users) #217

Merged
merged 1 commit into from
Jun 2, 2020

Conversation

lbalmaceda
Copy link
Contributor

Changes

This method was added 5 years ago. It seems it is no longer available nor documented on the API2 docs. This PR prints a warning to whoever is still calling it.

The use of the method would result on a 404 response from the server.

References

https://auth0.com/docs/api/management/v2#!/Users

@lbalmaceda lbalmaceda added the tiny Tiny review label May 29, 2020
@lbalmaceda lbalmaceda requested a review from a team May 29, 2020 18:17
@lbalmaceda lbalmaceda added this to the v3-Next milestone May 29, 2020
Copy link
Contributor

@adamjmcgrath adamjmcgrath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lbalmaceda
Copy link
Contributor Author

It might be (that change you linked dates from 2016, good find anyway 🔍 ). But still, none of them is documented in the API2 swagger docs. If an endpoint is not published there, I'd think it's either not available or not intended to be used.

The list of available User endpoints:
image

@lbalmaceda
Copy link
Contributor Author

It seems not possible since 2018. https://community.auth0.com/t/is-there-a-way-to-bulk-delete-users/10832

@adamjmcgrath adamjmcgrath self-requested a review June 2, 2020 07:49
@lbalmaceda lbalmaceda merged commit af0c61b into master Jun 2, 2020
@lbalmaceda lbalmaceda deleted the fix-rm-all-users branch June 2, 2020 18:48
@lbalmaceda lbalmaceda modified the milestones: v3-Next, 3.11.0 Jun 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants