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

WebCryptoAPI: assert that CryptoKey instances have the expected Symbol.toStringTag #37716

Merged
merged 2 commits into from
Jan 18, 2023
Merged

WebCryptoAPI: assert that CryptoKey instances have the expected Symbol.toStringTag #37716

merged 2 commits into from
Jan 18, 2023

Conversation

panva
Copy link
Contributor

@panva panva commented Jan 1, 2023

We've recently come across user's expectation that the Node.js implementation would have the same Symbol.toStringTag as browsers do for object type detection.

Refs: #37710
Refs: nodejs/node#45987

Having this WPT in place would have told us as implementers early that something we've missed is possibly already relied upon in the javascript ecosystem.

@panva
Copy link
Contributor Author

panva commented Jan 1, 2023

The change has not affected any of the wpt.fyi results.

Copy link
Member

@twiss twiss left a comment

Choose a reason for hiding this comment

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

Thanks!

WebCryptoAPI/util/helpers.js Outdated Show resolved Hide resolved
Co-authored-by: Daniel Huigens <d.huigens@protonmail.com>
@twiss twiss merged commit 450f829 into web-platform-tests:master Jan 18, 2023
jgerigmeyer added a commit to oddbird/wpt that referenced this pull request Jan 18, 2023
* master: (29 commits)
  WebCryptoAPI: check that CryptoKey instances have the expected Symbol.toStringTag (web-platform-tests#37716)
  ResizeObserver: Element with size 0x0 should be notified (web-platform-tests#38041)
  WebCryptoAPI: remove test for OperationError for tagLength 256 (web-platform-tests#37734)
  [scroll-timeline] Implement animation-range
  [Web Platform Test] Remove extra indent from shipping-option test
  [scroll-timeline] Reverse scroll-timeline-axis/name
  [Web Platform Test] Remove extra indents from delegations tests
  [Web Platform Test] Just-in-time install for shipping-option test
  [Web Platform Test] Just-in-time install for delegations test
  remove auth header from redirected cross-origin requests.
  Add tests for [Transferable] VideoFrame
  Make the style editor handle `<font>` at last when there are multiple preserved styles
  Make css-fonts/animations/font-variation-settings-composition.html less finicky about serialization order for font-variation-settings.
  [scroll-timeline] Hide support for tree-scoped timeline behind a flag
  Use script instead of fetch to de-flake firefox
  Retries in cache test
  Clean up doc test
  Fix CR comments
  Update preload/prefetch-cache.html
  Add test cases for cross-origin error events
  ...
@panva panva deleted the webcrypto-cryptokey-toStringTag branch January 18, 2023 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants