Skip to content
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.

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

feat: support for verifying ES256 and retrieving IAP public keys #887

Merged
merged 8 commits into from
Feb 12, 2020

Conversation

bshaffer
Copy link
Contributor

See samples/verifyIdToken-iap.js to see this in use.

Allows for the IAP public keys to be used to verify IAP-signed tokens. See this tutorial for more info

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 10, 2020
@bcoe bcoe added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 10, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 10, 2020
Copy link
Contributor

@bcoe bcoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like a good start to me, left some initial review.

I think it might also be worth updating the README, perhaps linking to the documentation you shared.

test/fixtures/ecdsa-private.pem Outdated Show resolved Hide resolved
src/auth/oauth2client.ts Show resolved Hide resolved
@bcoe bcoe added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 11, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 11, 2020
@codecov
Copy link

codecov bot commented Feb 11, 2020

Codecov Report

Merging #887 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #887   +/-   ##
=======================================
  Coverage   90.84%   90.84%           
=======================================
  Files          19       19           
  Lines        4218     4218           
  Branches      452      452           
=======================================
  Hits         3832     3832           
  Misses        380      380           
  Partials        6        6

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2529259...ee8ee2b. Read the comment docs.

@bshaffer
Copy link
Contributor Author

@bcoe I reviewed your changes, PTAL!

@bcoe bcoe merged commit a98e386 into googleapis:master Feb 12, 2020
@bshaffer bshaffer deleted the es256-verify branch February 12, 2020 20:50
gcf-merge-on-green bot pushed a commit that referenced this pull request Feb 14, 2020
This PR was generated using Autosynth. 🌈

Commits in this repo since last synth:
c532b14 chore(deps): update dependency linkinator to v2 (#886)
a98e386 feat: support for verifying ES256 and retrieving IAP public keys (#887)
e671629 build: add GitHub actions config for unit tests (#888)
163e43d test: modernize mocha config (#884)
58e5029 chore: skip img.shields.io in docs test
13e8cfb chore(deps): update dependency @types/mocha to v7
634cf6a chore: release 5.9.2 (#882)
63c4637 fix: populate credentials.refresh_token if provided (#881)


<details><summary>Log from Synthtool</summary>

```
synthtool > Executing /tmpfs/src/git/autosynth/working_repo/synth.py.
On branch autosynth
nothing to commit, working tree clean
HEAD detached at FETCH_HEAD
nothing to commit, working tree clean
.eslintignore
.eslintrc.yml
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.github/PULL_REQUEST_TEMPLATE.md
.github/release-please.yml
.github/workflows/ci.yaml
.jsdoc.js
.kokoro/common.cfg
.kokoro/continuous/node10/common.cfg
.kokoro/continuous/node10/docs.cfg
.kokoro/continuous/node10/lint.cfg
.kokoro/continuous/node10/samples-test.cfg
.kokoro/continuous/node10/system-test.cfg
.kokoro/continuous/node10/test.cfg
.kokoro/continuous/node12/common.cfg
.kokoro/continuous/node12/test.cfg
.kokoro/continuous/node8/common.cfg
.kokoro/continuous/node8/test.cfg
.kokoro/docs.sh
.kokoro/lint.sh
.kokoro/presubmit/node10/common.cfg
.kokoro/presubmit/node10/docs.cfg
.kokoro/presubmit/node10/lint.cfg
.kokoro/presubmit/node10/samples-test.cfg
.kokoro/presubmit/node10/system-test.cfg
.kokoro/presubmit/node10/test.cfg
.kokoro/presubmit/node12/common.cfg
.kokoro/presubmit/node12/test.cfg
.kokoro/presubmit/node8/common.cfg
.kokoro/presubmit/node8/test.cfg
.kokoro/presubmit/windows/common.cfg
.kokoro/presubmit/windows/test.cfg
.kokoro/publish.sh
.kokoro/release/docs.cfg
.kokoro/release/docs.sh
.kokoro/release/publish.cfg
.kokoro/samples-test.sh
.kokoro/system-test.sh
.kokoro/test.bat
.kokoro/test.sh
.kokoro/trampoline.sh
.nycrc
.prettierignore
.prettierrc
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
codecov.yaml
renovate.json
samples/README.md
synthtool > Wrote metadata to synth.metadata.

```
</details>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants