Bug 197771

Summary: [Flatpak] Use the safe PyYAML loader if available
Product: WebKit Reporter: Adrian Perez <aperez>
Component: Tools / TestsAssignee: Adrian Perez <aperez>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, calvaris, cgarcia, commit-queue, lforschler, pnormand, tsaunier, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Adrian Perez
Reported 2019-05-10 06:39:32 PDT
Current versions of PyYAML give the following warning: /home/aperez/devel/WebKit/Tools/flatpak/flatpakutils.py:159: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details. manifest = yaml.load(contents) Let's fix this and use “yaml.safe_load()” whenever it's available.
Attachments
Patch (1.46 KB, patch)
2019-05-10 06:42 PDT, Adrian Perez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adrian Perez
Comment 1 2019-05-10 06:42:21 PDT
Created attachment 369551 [details] Patch
WebKit Commit Bot
Comment 2 2019-05-10 07:42:33 PDT
Comment on attachment 369551 [details] Patch Clearing flags on attachment: 369551 Committed r245172: <https://trac.webkit.org/changeset/245172>
WebKit Commit Bot
Comment 3 2019-05-10 07:42:34 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 4 2019-05-10 07:43:21 PDT
<rdar://problem/50662814>
Note You need to log in before you can comment on or make changes to this bug.