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.

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

feat: adds crypto utils for AWS request signing #1046

Merged
merged 15 commits into from
Aug 24, 2020
Merged

feat: adds crypto utils for AWS request signing #1046

merged 15 commits into from
Aug 24, 2020

Conversation

bojeil-google
Copy link
Contributor

Utils added to both browser and Node.js crypto.

grayside and others added 14 commits August 7, 2020 16:32
@grayside
refactor(samples): idtoken-cloudrun => idtoken-serverless (#1025)
11b0cb7
@yoshi-automation
docs: README updated with new sample paths
4df48df
@release-please
chore: release 6.0.7 (#1023)
6dfee1d 
* chore: updated samples/package.json [ci skip]

* chore: updated CHANGELOG.md [ci skip]

* chore: updated package.json

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
@yoshi-automation
build: credential-file-override is no longer required (#1029)
b976c8a 
This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/5f7f9c6d-c75a-4c60-8bb8-0026a14cead7/targets

- [ ] To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@94421c4
@yoshi-automation
chore: update cloud rad kokoro build job (#1034)
4830a53 
This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/b742586e-df31-4aac-8092-78288e9ea8e7/targets

- [ ] To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@bd0deaa
@renovate-bot
fix(deps): roll back dependency google-auth-library to ^6.0.6 (#1033)
eb54ee9
@SurferJeffAtGoogle
build: perform publish using Node 12 (#1035)
714247d 
This PR was generated using Autosynth. 🌈



- [ ] To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@5747555
@release-please
chore: release 6.0.8 (#1036)
211e042 
* chore: updated samples/package.json [ci skip]

* chore: updated CHANGELOG.md [ci skip]

* chore: updated package.json

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
@xethlyx
docs: correct spelling of its (#1040)
d4e56c0 
Minor change that fixes a spelling error. I don't think an issue needed to be opened for this. Thanks!
@banyan @JustinBeckwith
@sofisl @bcoe
docs: fix usage in readme (#1031)
6c15139 
Add await

Co-authored-by: Justin Beckwith <justin.beckwith@gmail.com>
Co-authored-by: sofisl <55454395+sofisl@users.noreply.github.com>
Co-authored-by: Benjamin E. Coe <bencoe@google.com>
@yoshi-automation
chore: start tracking obsolete files (#1043)
0c8e086 
This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/7a1b0b96-8ddb-4836-a1a2-d2f73b7e6ffe/targets

- [ ] To automatically regenerate this PR, check this box.
@yoshi-automation
build: move system and samples test from Node 10 to Node 12 (#1045)
5164845 
This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/ba2d388f-b3b2-4ad7-a163-0c6b4d86894f/targets

- [ ] To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@05de3e1
@bojeil-google
Merge remote-tracking branch 'upstream/byoid'
1b19c16
@bojeil-google
feat: adds crypto utils for AWS request signing
81f94ac 
Utils added to both browser and Node.js crypto.
@bojeil-google bojeil-google requested a review from a team as a code owner August 21, 2020 23:00
@codecov
Copy link

codecov bot commented Aug 21, 2020 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (byoid@bbcd03d). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##             byoid    #1046   +/-   ##
========================================
  Coverage         ?   92.50%           
========================================
  Files            ?       25           
  Lines            ?     5244           
  Branches         ?      588           
========================================
  Hits             ?     4851           
  Misses           ?      393           
  Partials         ?        0

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 bbcd03d...b36504c. Read the comment docs.

@google-cla
Copy link

google-cla bot commented Aug 21, 2020

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added the cla: no This human has *not* signed the Contributor License Agreement. label Aug 21, 2020
@google-cla
Copy link

google-cla bot commented Aug 21, 2020

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

@bcoe bcoe added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels Aug 24, 2020
@bcoe bcoe merged commit 26ab5e3 into googleapis:byoid Aug 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bcoe bcoe bcoe approved these changes

Assignees

@sofisl sofisl

Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@bojeil-google @bcoe @sofisl @grayside @yoshi-automation @renovate-bot @SurferJeffAtGoogle @xethlyx @banyan