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

ASSERTION FAILED: !m_shouldFilterICECandidates || sdp.contains(".local"_s) || sdp.contains(" srflx "_s) in walmart.com #13952

Conversation

youennf
Copy link
Contributor

@youennf youennf commented May 17, 2023

d19ee7d

ASSERTION FAILED: !m_shouldFilterICECandidates || sdp.contains(".local"_s) || sdp.contains(" srflx "_s) in walmart.com
https://bugs.webkit.org/show_bug.cgi?id=256854
rdar://109417446

Reviewed by Eric Carlson.

Updated the debug assert to account for relay candidates which can always be exposed.

* Source/WebCore/Modules/mediastream/PeerConnectionBackend.cpp:
(WebCore::PeerConnectionBackend::newICECandidate):

Canonical link: https://commits.webkit.org/264155@main

a659d7b

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style βœ… πŸ›  ios βœ… πŸ›  mac βœ… πŸ›  wpe βœ… πŸ›  wincairo
βœ… πŸ§ͺ bindings βœ… πŸ›  ios-sim βœ… πŸ›  mac-AS-debug βœ… πŸ§ͺ wpe-wk2
βœ… πŸ§ͺ webkitperl βœ… πŸ§ͺ ios-wk2 βœ… πŸ§ͺ api-mac βœ… πŸ›  gtk
βœ… πŸ§ͺ ios-wk2-wpt βœ… πŸ§ͺ mac-wk1 ❌ πŸ§ͺ gtk-wk2
βœ… πŸ§ͺ api-ios βœ… πŸ§ͺ mac-wk2 βœ… πŸ§ͺ api-gtk
βœ… πŸ›  tv βœ… πŸ§ͺ mac-AS-debug-wk2
βœ… πŸ›  tv-sim
βœ… πŸ›  πŸ§ͺ merge βœ… πŸ›  watch
βœ… πŸ›  watch-sim

@youennf youennf self-assigned this May 17, 2023
@youennf youennf added the WebRTC For bugs in WebRTC label May 17, 2023
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label May 17, 2023
@youennf youennf removed the merging-blocked Applied to prevent a change from being merged label May 17, 2023
@youennf youennf added the merge-queue Applied to send a pull request to merge-queue label May 17, 2023
…l"_s) || sdp.contains(" srflx "_s) in walmart.com

https://bugs.webkit.org/show_bug.cgi?id=256854
rdar://109417446

Reviewed by Eric Carlson.

Updated the debug assert to account for relay candidates which can always be exposed.

* Source/WebCore/Modules/mediastream/PeerConnectionBackend.cpp:
(WebCore::PeerConnectionBackend::newICECandidate):

Canonical link: https://commits.webkit.org/264155@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/ASSERTION-FAILED-m_shouldFilterICECandidates--sdp-contains-local_s--sdp-contains-srflx-_s-in-walmart-com branch from a659d7b to d19ee7d Compare May 17, 2023 17:07
@webkit-commit-queue
Copy link
Collaborator

Committed 264155@main (d19ee7d): https://commits.webkit.org/264155@main

Reviewed commits have been landed. Closing PR #13952 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit d19ee7d into WebKit:main May 17, 2023
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WebRTC For bugs in WebRTC
Projects
None yet
5 participants