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.

Sign up for GitHub

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

Jump to bottom

Require auth token but skip verification for tasks queue functions. #1073

Merged

taeold merged 2 commits into master from dl-tq-auth

Apr 12, 2022

Contributor

taeold commented Apr 11, 2022 •

edited

Previously, we allowed TQ functions to either:

Not have any auth token
Have admin SDK verified auth tokens.

Since TQ functions are guarded by IAM, we instead implement the following behavior:

MUST have auth token
Do not verify auth token - instead we just decode it and pass it to the handler


          Skip auth token verification for tasks queue functions.

15abd26

taeold requested review from inlined and colerogers

April 11, 2022 22:05


          Add changelog.

53b22d4

inlined approved these changes

View reviewed changes

taeold merged commit 60a7a40 into master

taeold deleted the dl-tq-auth branch

April 12, 2022 21:11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment