Skip to content

Fix HkdfParams use and remove HkdfCtrParams #879

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

Merged
merged 2 commits into from
Sep 9, 2020
Merged

Conversation

saghul
Copy link
Contributor

@saghul saghul commented Jun 30, 2020

The latter was removed in 2016: w3c/webcrypto#16 with the former being in the current spec: https://w3c.github.io/webcrypto/#hkdf-params

@saghul saghul requested a review from sandersn as a code owner June 30, 2020 14:10
@ghost
Copy link

ghost commented Jun 30, 2020

CLA assistant check
All CLA requirements met.

@uhoreg
Copy link

uhoreg commented Jul 4, 2020

Coincidentally, I ran into this issue earlier today and came up with essentially the same fix before noticing this PR.

HkdfCtrParams was replaced with HkdfParams in the December 2016 WebCrypto Proposed Recommendation, so HkdfCtrParams is obsolete by at least 3.5 years.

@saghul
Copy link
Contributor Author

saghul commented Sep 8, 2020

Gentle ping, @sandersn

@orta orta mentioned this pull request Sep 9, 2020
@orta
Copy link
Contributor

orta commented Sep 9, 2020

Looks good, thanks!

@orta orta merged commit f7b3770 into microsoft:master Sep 9, 2020
benknab added a commit to benknab/tarnhelm that referenced this pull request Dec 6, 2020
benknab added a commit to benknab/tarnhelm that referenced this pull request Dec 6, 2020
* Bump packages

Fixes security issues in object-path and node-fetch

* Add _static to prettier ignore

* Update code style to comply with new ESLint rules

* Update crypto types

See: microsoft/TypeScript-DOM-lib-generator#879

* Fix WebSocket async listen function response type
benknab added a commit to benknab/tarnhelm that referenced this pull request Dec 6, 2020
* Bump packages to fix security issues (#17)

* Bump packages

Fixes security issues in object-path and node-fetch

* Add _static to prettier ignore

* Update code style to comply with new ESLint rules

* Update crypto types

See: microsoft/TypeScript-DOM-lib-generator#879

* Fix WebSocket async listen function response type

* Update check scripts and configs

* Apply new style, lint, type rules
benknab added a commit to benknab/tarnhelm that referenced this pull request Jan 11, 2021
* Add ln-service

* Add LND config

* Add basic lightning network payment integration

* Add CoinGecko to query bitcoin price

* Update upload params validation

* Add price quote calculation

* Apply eslint override to one line only

* Add file limit equal to paid size

* Fix empty file meta

* Add LN invoice QR code

* Add price estimate row

* Fix wrapped glitch effect in button

* Add invoice, invoice update data restriction

* Merge package update with lnd (#18)

* Bump packages to fix security issues (#17)

* Bump packages

Fixes security issues in object-path and node-fetch

* Add _static to prettier ignore

* Update code style to comply with new ESLint rules

* Update crypto types

See: microsoft/TypeScript-DOM-lib-generator#879

* Fix WebSocket async listen function response type

* Update check scripts and configs

* Apply new style, lint, type rules

* Add pino logger
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants