Closed
Bug 1688438
Opened 3 years ago
Closed 2 years ago
Replace ws in WebRender debugger with another websocket implementation
Categories
(Core :: Graphics: WebRender, enhancement)
Core
Graphics: WebRender
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: xidorn, Unassigned)
References
Details
(Keywords: good-first-bug)
ws package is mostly unmaintained, and contains an open vulnerability. While it's probably not a big concern for WebRender as it's only used in debugger, it is still better to replace it with something else.
|
||
Updated•3 years ago
|
Severity: -- → N/A
Keywords: good-first-bug
|
||
Comment 1•3 years ago
|
||
Can i work on this? I could replace it with any other popular implementation after discussion
Flags: needinfo?(xidorn+moz)
Flags: needinfo?(jmathies)
|
Reporter | |
Comment 2•3 years ago
|
||
I'll leave the decision to Jim.
It seems the easiest thing to do is probably to replace with the parity fork parity-ws. Alternative, we can switch to the more popular tungstenite, but it's unclear how much effort it would take.
Flags: needinfo?(xidorn+moz)
|
|
||
Comment 3•3 years ago
|
||
Okay i will wait.
|
||
Updated•3 years ago
|
Blocks: gfx-triage
Flags: needinfo?(jmathies)
|
||
Comment 4•3 years ago
|
||
The parity fork seems fine.
|
|
||
Updated•3 years ago
|
No longer blocks: gfx-triage
|
||
Comment 5•2 years ago
|
||
ws is no longer used in WR.
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•