btwedutech/hoppscotch
5
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 6 Wiki Activity

hotfix: refresh token cookie migration (HBE-167) (#31)

Browse Source 
* chore: removed signed cookies from refresh token strategy

* chore: removed signed cookies from refresh RTCookie decorator
This commit is contained in:
Balu Babu
2023-03-03 15:52:26 +05:30
committed by GitHub
parent a938be3712
commit 80c6f600db
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
packages/hoppscotch-backend/src/auth/strategies/rt-jwt.strategy.ts
View File

@@ -21,7 +21,7 @@ export class RTJwtStrategy extends PassportStrategy(Strategy, 'jwt-refresh') {
super({ super({
jwtFromRequest: ExtractJwt.fromExtractors([ jwtFromRequest: ExtractJwt.fromExtractors([
(request: Request) => { (request: Request) => {
const RTCookie = request.signedCookies['refresh_token']; const RTCookie = request.cookies['refresh_token'];
if (!RTCookie) { if (!RTCookie) {
throw new ForbiddenException(COOKIES_NOT_FOUND); throw new ForbiddenException(COOKIES_NOT_FOUND);
} }

2
packages/hoppscotch-backend/src/decorators/rt-cookie.decorator.ts
View File

@@ -7,6 +7,6 @@ import { GqlExecutionContext } from '@nestjs/graphql';
export const RTCookie = createParamDecorator( export const RTCookie = createParamDecorator(
(data: unknown, context: ExecutionContext) => { (data: unknown, context: ExecutionContext) => {
const ctx = GqlExecutionContext.create(context); const ctx = GqlExecutionContext.create(context);
return ctx.getContext().req.signedCookies['refresh_token']; return ctx.getContext().req.cookies['refresh_token'];
}, },
); );