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

[Gecko Bug 1554087] [wpt PR 16993] - webrtc wpt: validate connectionstate goes to failed with wrong fingerprints, a=testonly #17466

Merged
merged 1 commit into from Jul 24, 2019

Conversation

moz-wptsync-bot
Copy link
Collaborator

Automatic update from web-platform-tests
webrtc wpt: validate connectionstate goes to failed with wrong fingerprints

Modifies the SDP to pass an different fingerprint from the one that is used which
results in a DTLS failure/alert as the fingerprint does not match the certificate.

Bug: none
Change-Id: Ice8adf70f57e7da02fc014bb43b0c78841bfd9aa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1627362
Commit-Queue: Philipp Hancke philipp.hancke@googlemail.com
Reviewed-by: Henrik Boström hbos@chromium.org
Cr-Commit-Position: refs/heads/master@{#663037}

--

wp5At-commits: 80853e0
wpt-pr: 16993

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1554087
gecko-commit: f5fd810645175d8996685aebb59fd0417df92dfb
gecko-integration-branch: central
gecko-reviewers: testonly

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.

Already reviewed downstream.

…with wrong fingerprints, a=testonly

Automatic update from web-platform-tests
webrtc wpt: validate connectionstate goes to failed with wrong fingerprints

Modifies the SDP to pass an different fingerprint from the one that is used which
results in a DTLS failure/alert as the fingerprint does not match the certificate.

Bug: none
Change-Id: Ice8adf70f57e7da02fc014bb43b0c78841bfd9aa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1627362
Commit-Queue: Philipp Hancke <philipp.hancke@googlemail.com>
Reviewed-by: Henrik Boström <hbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#663037}

--

wp5At-commits: 80853e0
wpt-pr: 16993

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1554087
gecko-commit: f5fd810645175d8996685aebb59fd0417df92dfb
gecko-integration-branch: central
gecko-reviewers: testonly
@jgraham
Copy link
Contributor

jgraham commented Jul 24, 2019

Looks like the runtime of the stability task was too long.

@jgraham jgraham merged commit a652b55 into master Jul 24, 2019
@gsnedders gsnedders deleted the gecko/1554087 branch September 10, 2019 17:23
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.

None yet

4 participants