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

webrtc wpt: fix generateCertificates WPT, add one for rejecting SHA-1 and delete legacy tests #48200

Merged
merged 1 commit into from
Sep 16, 2024

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Sep 16, 2024

  • Fixes the two remaining WPT failures for generateCertificate.
  • Adds a negative test for SHA-1 for RSA certificates since SHA-1 should not be used for this anymore
  • clean up legacy fast/ generateCertificate tests which are superseded by the WPT ones.

BUG=webrtc:364338811

Change-Id: I513434b4084a2852dde7c340c8760640f3e25cb3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5863274
Reviewed-by: Henrik Boström <hbos@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Commit-Queue: Philipp Hancke <philipp.hancke@googlemail.com>
Cr-Commit-Position: refs/heads/main@{#1355957}

… and delete legacy tests

* Fixes the two remaining WPT failures for generateCertificate.
* Adds a negative test for SHA-1 for RSA certificates since SHA-1 should not be used for this anymore
* clean up legacy fast/ generateCertificate tests which are superseded by the WPT ones.

BUG=webrtc:364338811

Change-Id: I513434b4084a2852dde7c340c8760640f3e25cb3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5863274
Reviewed-by: Henrik Boström <hbos@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Commit-Queue: Philipp Hancke <philipp.hancke@googlemail.com>
Cr-Commit-Position: refs/heads/main@{#1355957}
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot merged commit bced097 into master Sep 16, 2024
18 checks passed
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-5863274 branch September 16, 2024 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment