Skip to content

Commit

Permalink
fix: add credentials header to refresh token call
Browse files Browse the repository at this point in the history
  • Loading branch information
jonathonadams committed Aug 21, 2020
1 parent a504b98 commit 83d2776
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
4 changes: 4 additions & 0 deletions dev-ops/cloudbuild.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,10 @@ steps:
args:
- push
- gcr.io/$PROJECT_ID/$_PROJECT_NAME:$TAG_NAME
- name: gcr.io/cloud-builders/docker
args:
- push
- gcr.io/$PROJECT_ID/$_PROJECT_NAME:latest

# Trigger a rolling update for the cluster
- name: gcr.io/cloud-builders/kubectl
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,7 @@ export class AuthService {
{
headers: this.headers,
reportProgress: false,
withCredentials: true,
}
);
}
Expand Down
2 changes: 0 additions & 2 deletions libs/server/auth/src/lib/routes/routes.ts
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,6 @@ export function refreshTokenRoute<R extends Refresh>(
(ctx.request as any).body.refreshToken ||
ctx.cookies.get('refresh_token');

if (!refreshToken) throw Boom.unauthorized('No token provided');

ctx.body = await refreshAccessTokenCtr(refreshToken, ctx.cookies);
};
}
Expand Down

0 comments on commit 83d2776

Please sign in to comment.