fix(authentication): Omit query in JWT strategy (#2011)

feathersjs · Jul 12, 2020 · 04ce7e9 · 04ce7e9
1 parent ebd8987
commit 04ce7e9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/packages/authentication/src/jwt.ts b/packages/authentication/src/jwt.ts
@@ -94,7 +94,7 @@ export class JWTStrategy extends AuthenticationBaseStrategy {
       throw new NotAuthenticated(`Could not find entity service`);
     }
 
-    const result = await entityService.get(id, omit(params, 'provider'));
+    const result = await entityService.get(id, omit(params, 'provider', 'query'));
 
     if (!params.provider) {
       return result;

diff --git a/packages/authentication/test/jwt.test.ts b/packages/authentication/test/jwt.test.ts
@@ -75,10 +75,14 @@ describe('authentication/jwt', () => {
     app.setup();
   });
 
-  it('getEntity', async () => {
+  it('getEntity (and params.query)', async () => {
     const [ strategy ] = app.service('authentication').getStrategies('jwt') as JWTStrategy[];
 
-    let entity = await strategy.getEntity(user.id, {});
+    let entity = await strategy.getEntity(user.id, {
+      query: {
+        name: 'Dave'
+      }
+    });
 
     assert.deepStrictEqual(entity, user);