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

Apply credential stripping to all untransforms for _User #1498

Merged

Conversation

TylerBrock
Copy link
Contributor

@TylerBrock TylerBrock commented Apr 14, 2016

The tests describe what is going on. Basically we also want to strip sessionToken and auth data when isMaster is used. Fixes #1482

@facebook-github-bot
Copy link

@TylerBrock updated the pull request.

@codecov-io
Copy link

Current coverage is 92.79%

Merging #1498 into master will decrease coverage by -0.09% as of 7b2c0b0

@@            master   #1498   diff @@
======================================
  Files           87      87       
  Stmts         5496    5496       
  Branches      1017    1017       
  Methods          0       0       
======================================
- Hit           5105    5100     -5
  Partial         10      10       
- Missed         381     386     +5

Review entire Coverage Diff as of 7b2c0b0

Powered by Codecov. Updated on successful CI builds.

@drew-gross
Copy link
Contributor

Good catch! Looks good to me.

@drew-gross drew-gross merged commit d57e384 into parse-community:master Apr 14, 2016
steven-supersolid added a commit to supersolid/parse-server that referenced this pull request Apr 20, 2016
steven-supersolid added a commit to supersolid/parse-server that referenced this pull request Apr 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants