RESOLVED FIXED 238041
Sandbox: Remove telemetry in Network Process sandbox macOS 
https://bugs.webkit.org/show_bug.cgi?id=238041
Summary Sandbox: Remove telemetry in Network Process sandbox macOS
Adam Mazander
Reported 2022-03-17 13:07:13 PDT
We have enough data, removing telemetry
Attachments
Patch (21.94 KB, patch)
2022-03-17 13:14 PDT, Adam Mazander 		no flags
DetailsFormatted DiffDiff
Patch (21.94 KB, patch)
2022-03-17 13:35 PDT, Adam Mazander 		no flags
DetailsFormatted DiffDiff
Patch (6.21 KB, patch)
2022-03-17 13:59 PDT, Adam Mazander 		no flags
DetailsFormatted DiffDiff
Patch (6.31 KB, patch)
2022-03-18 10:13 PDT, Adam Mazander 		no flags
DetailsFormatted DiffDiff
Patch (6.03 KB, patch)
2022-03-18 12:57 PDT, Adam Mazander 		no flags
DetailsFormatted DiffDiff
Patch (5.56 KB, patch)
2022-03-18 17:10 PDT, Adam Mazander 		no flags
DetailsFormatted DiffDiff
Patch (5.17 KB, patch)
2022-03-18 19:37 PDT, Adam Mazander 		no flags
DetailsFormatted DiffDiff
Show Obsolete (6) View All Add attachment proposed patch, testcase, etc.
Adam Mazander
Comment 1 2022-03-17 13:09:02 PDT
<rdar://90448244>
Adam Mazander
Comment 2 2022-03-17 13:14:28 PDT
Created attachment 455013 [details] Patch
Adam Mazander
Comment 3 2022-03-17 13:35:20 PDT
Created attachment 455018 [details] Patch
Adam Mazander
Comment 4 2022-03-17 13:59:36 PDT
Created attachment 455024 [details] Patch
Per Arne Vollan
Comment 5 2022-03-17 14:03:00 PDT
Comment on attachment 455024 [details] Patch R=me.
Per Arne Vollan
Comment 6 2022-03-18 09:40:57 PDT
Comment on attachment 455024 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=455024&action=review > Source/WebKit/NetworkProcess/mac/com.apple.WebKit.NetworkProcess.sb.in:-57 > -(allow file-map-executable (with telemetry)) Removed one too many parentheses. > Source/WebKit/NetworkProcess/mac/com.apple.WebKit.NetworkProcess.sb.in:-165 > -(deny sysctl* (with telemetry)) Ditto.
Adam Mazander
Comment 7 2022-03-18 10:05:43 PDT
adding back telemetry for file-map-executable
Adam Mazander
Comment 8 2022-03-18 10:13:18 PDT
Created attachment 455106 [details] Patch
Per Arne Vollan
Comment 9 2022-03-18 10:19:40 PDT
Comment on attachment 455106 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=455106&action=review > Source/WebKit/NetworkProcess/mac/com.apple.WebKit.NetworkProcess.sb.in:165 > -(deny sysctl* (with telemetry)) > +(deny sysctl* Missing parenthesis. > Source/WebKit/NetworkProcess/mac/com.apple.WebKit.NetworkProcess.sb.in:179 > + (sysctl-name-prefix "net.routetable"))) Remove last parenthesis.
Adam Mazander
Comment 10 2022-03-18 12:57:23 PDT
Created attachment 455127 [details] Patch
Per Arne Vollan
Comment 11 2022-03-18 16:48:43 PDT
Comment on attachment 455127 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=455127&action=review > Source/WebKit/NetworkProcess/mac/com.apple.WebKit.NetworkProcess.sb.in:464 > - (allow syscall-unix (with telemetry)) > + (allow syscall-unix) We may want to keep this a little longer. > Source/WebKit/NetworkProcess/mac/com.apple.WebKit.NetworkProcess.sb.in:594 > - (allow mach-message-send (with telemetry))))) > + (allow mach-message-send )))) Ditto. > Source/WebKit/NetworkProcess/mac/com.apple.WebKit.NetworkProcess.sb.in:597 > - (allow syscall-mach (with report) (with telemetry)) > + (allow syscall-mach (with report)) Ditto.
Adam Mazander
Comment 12 2022-03-18 17:10:42 PDT
Created attachment 455153 [details] Patch
Adam Mazander
Comment 13 2022-03-18 17:11:28 PDT
added back syscall telemetry
Per Arne Vollan
Comment 14 2022-03-18 18:19:02 PDT
Comment on attachment 455153 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=455153&action=review > Source/WebKit/NetworkProcess/mac/com.apple.WebKit.NetworkProcess.sb.in:-597 > - (allow syscall-mach (with report) (with telemetry)) There seems to be missing a paranthesis here.
Adam Mazander
Comment 15 2022-03-18 19:37:51 PDT
Created attachment 455160 [details] Patch
Adam Mazander
Comment 16 2022-03-18 19:40:19 PDT
Fixed. Thanks.
Brent Fulgham
Comment 17 2022-03-21 10:31:57 PDT
Comment on attachment 455160 [details] Patch r=me
EWS
Comment 18 2022-03-21 10:38:20 PDT
Committed r291563 (248665@main): <https://commits.webkit.org/248665@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 455160 [details].
Note You need to log in before you can comment on or make changes to this bug.