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

Makes sure we don't strip authData or session token from users using masterKey #2348

Merged
merged 2 commits into from
Jul 23, 2016

Conversation

flovilmart
Copy link
Contributor

@flovilmart flovilmart commented Jul 20, 2016

When querying for Auth, the RestQuery would strip out sessionToken and authData from the response. This fix makes sure we keep the authData and the session token when using the masterKey so the user is fully available in cloud code.

Fixes #2342

@ghost
Copy link

ghost commented Jul 20, 2016

@flovilmart updated the pull request.

@codecov-io
Copy link

codecov-io commented Jul 20, 2016

Current coverage is 91.81%

Merging #2348 into master will not change coverage

@@             master      #2348   diff @@
==========================================
  Files            94         94          
  Lines         10479      10479          
  Methods        1275       1275          
  Messages          0          0          
  Branches       1713       1713          
==========================================
  Hits           9621       9621          
  Misses          858        858          
  Partials          0          0          

Powered by Codecov. Last updated by ab3b4ad...492c4cb

@flovilmart flovilmart added this to the 2.2.17 milestone Jul 22, 2016
@TylerBrock TylerBrock merged commit c9fc809 into master Jul 23, 2016
@flovilmart flovilmart deleted the fix/2342-cloud-code-user branch August 20, 2016 20:06
rsouzas pushed a commit to back4app/parse-server that referenced this pull request Mar 15, 2017
…masterKey (parse-community#2348)

* Makes sure we don't strip auth data or session token from users queried with masterKey (parse-community#2342))

* nit: test title
rsouzas pushed a commit to back4app/parse-server that referenced this pull request Mar 16, 2017
…masterKey (parse-community#2348)

* Makes sure we don't strip auth data or session token from users queried with masterKey (parse-community#2342))

* nit: test title
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.

Parse.FacebookUtils.isLinked(user) returns false since parse-server 2.2.6
4 participants