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

User field authData cannot be deleted #1957

Open
4 tasks done
mtrezza opened this issue Dec 10, 2021 · 3 comments
Open
4 tasks done

User field authData cannot be deleted #1957

mtrezza opened this issue Dec 10, 2021 · 3 comments
Labels
bounty:$10 Bounty applies for fixing this issue (Parse Bounty Program) type:bug Impaired feature or lacking behavior that is likely assumed

Comments

@mtrezza
Copy link
Member

mtrezza commented Dec 10, 2021

New Issue Checklist

Issue Description

Trying to delete the authData field in the _User collection fails with error:

This authentication method is unsupported.

Steps to reproduce

  1. Find user with a populated authData field
  2. Select cell
  3. Hit delete button

Actual Outcome

Error message appears, field value is not deleted.

Expected Outcome

Field value should be deleted.

Note

It also does not work to only select a specific auth provider in the authData field and delete on that provider entry, e.g. "facebook: { ... }" or "apple: { ... }". The result appears correctly in the cell with the authData field updated to {}, even showing a success pop-up. But after refreshing the data browser, the auth data is still there.

Environment

Dashboard

  • Parse Dashboard version: 3.2.1
@parse-github-assistant
Copy link

parse-github-assistant bot commented Dec 10, 2021

Thanks for opening this issue!

  • 🚀 You can help us to fix this issue faster by opening a pull request with a failing test. See our Contribution Guide for how to make a pull request, or read our New Contributor's Guide if this is your first time contributing.

@mtrezza mtrezza added severity:low type:bug Impaired feature or lacking behavior that is likely assumed bounty:$10 Bounty applies for fixing this issue (Parse Bounty Program) labels Dec 10, 2021
@sadakchap
Copy link
Member

@mtrezza I'm not sure whether we can set authData back to null.

Please refer this testcase in parse-server.

@mtrezza
Copy link
Member Author

mtrezza commented Dec 26, 2021

I've added that test case in parse-community/parse-server#6154 because there is no actual authData field in the DB. What we see in the dashboard as authData is synthesized of all individual _auth_data_<provider> fields. Creating an actual authData field in the DB of value null would crash the server, hence the PR.

However, if one deletes the whole authData field in dashboard the appropriate action would be to delete all individual _auth_data_<provider> fields in the DB. It seems that this logic is currently missing, possibly only in Parse Server and this may not even be a Parse Dashboard issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bounty:$10 Bounty applies for fixing this issue (Parse Bounty Program) type:bug Impaired feature or lacking behavior that is likely assumed
Projects
None yet
Development

No branches or pull requests

2 participants