RESOLVED FIXED 209478
[GTK][WPE] WEB_PROCESS_CMD_PREFIX and others have stopped working 
https://bugs.webkit.org/show_bug.cgi?id=209478
Summary [GTK][WPE] WEB_PROCESS_CMD_PREFIX and others have stopped working
Chris Lord
Reported 2020-03-24 08:14:20 PDT
This was caused by the switch to Flatpak dependencies, which now sandboxes the environment. These variables need to be white-listed.
Attachments
Patch (1.40 KB, patch)
2020-03-24 08:19 PDT, Chris Lord 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Lord
Comment 1 2020-03-24 08:19:24 PDT
Created attachment 394367 [details] Patch
Philippe Normand
Comment 2 2020-03-24 08:22:21 PDT
Comment on attachment 394367 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=394367&action=review > Tools/ChangeLog:6 > + Make sure *_PROCESS_CMD_PREFIX variables are whitelisted in the Flatpak sandbox. An alternative could have been to add this to `env_var_suffixes_to_keep` but I'm also OK with this approach. Explicit is better than implicit usually :)
Chris Lord
Comment 3 2020-03-24 08:25:26 PDT
Comment on attachment 394367 [details] Patch I agree with explicit being better (though maybe I'll disagree when this comes back and bites us again ;)) so leaving as is. It would be good if these variables had 'WEBKIT' somewhere in them I think, if we were going to whitelist the suffix.
Philippe Normand
Comment 4 2020-03-24 08:37:46 PDT
Yeah, or a WEBKIT_ prefix :)
EWS
Comment 5 2020-03-24 08:46:52 PDT
Committed r258910: <https://trac.webkit.org/changeset/258910> All reviewed patches have been landed. Closing bug and clearing flags on attachment 394367 [details].
Radar WebKit Bug Importer
Comment 6 2020-03-24 08:47:14 PDT
<rdar://problem/60826356>
Note You need to log in before you can comment on or make changes to this bug.