Skip to content

Commit ea07eb5

Browse files
committed
Clears session on password change
- Fixes error type when passing an invalid session token
1 parent cf7202f commit ea07eb5

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

spec/ParseUser.spec.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1606,7 +1606,9 @@ describe('Parse.User testing', () => {
16061606
}).then(function(newUser) {
16071607
fail('Session should have been invalidated');
16081608
done();
1609-
}, function() {
1609+
}, function(err) {
1610+
expect(err.code).toBe(209);
1611+
expect(err.message).toBe('invalid session token');
16101612
done();
16111613
});
16121614
});

src/RestWrite.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -306,7 +306,7 @@ RestWrite.prototype.transformUser = function() {
306306
if (!this.data.password) {
307307
return;
308308
}
309-
if (this.query) {
309+
if (this.query && !this.auth.isMaster ) {
310310
this.storage['clearSessions'] = true;
311311
}
312312
return passwordCrypto.hash(this.data.password).then((hashedPassword) => {

src/Routers/UsersRouter.js

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,7 @@ export class UsersRouter extends ClassesRouter {
4141

4242
handleMe(req) {
4343
if (!req.info || !req.info.sessionToken) {
44-
throw new Parse.Error(Parse.Error.OBJECT_NOT_FOUND,
45-
'Object not found.');
44+
throw new Parse.Error(Parse.Error.INVALID_SESSION_TOKEN, 'invalid session token');
4645
}
4746
return rest.find(req.config, Auth.master(req.config), '_Session',
4847
{ _session_token: req.info.sessionToken },
@@ -51,8 +50,7 @@ export class UsersRouter extends ClassesRouter {
5150
if (!response.results ||
5251
response.results.length == 0 ||
5352
!response.results[0].user) {
54-
throw new Parse.Error(Parse.Error.OBJECT_NOT_FOUND,
55-
'Object not found.');
53+
throw new Parse.Error(Parse.Error.INVALID_SESSION_TOKEN, 'invalid session token');
5654
} else {
5755
let user = response.results[0].user;
5856
return { response: user };
@@ -145,10 +143,10 @@ export class UsersRouter extends ClassesRouter {
145143
let router = new PromiseRouter();
146144
router.route('GET', '/users', req => { return this.handleFind(req); });
147145
router.route('POST', '/users', req => { return this.handleCreate(req); });
146+
router.route('GET', '/users/me', req => { return this.handleMe(req); });
148147
router.route('GET', '/users/:objectId', req => { return this.handleGet(req); });
149148
router.route('PUT', '/users/:objectId', req => { return this.handleUpdate(req); });
150149
router.route('DELETE', '/users/:objectId', req => { return this.handleDelete(req); });
151-
router.route('GET', '/users/me', req => { return this.handleMe(req); });
152150
router.route('GET', '/login', req => { return this.handleLogIn(req); });
153151
router.route('POST', '/logout', req => { return this.handleLogOut(req); });
154152
router.route('POST', '/requestPasswordReset', () => {

0 commit comments

Comments
 (0)