RESOLVED FIXED 221111
[webkitcrepy] Handle case where pypi serves invalid html 
https://bugs.webkit.org/show_bug.cgi?id=221111
Summary [webkitcrepy] Handle case where pypi serves invalid html
Jonathan Bedard
Reported 2021-01-28 16:51:08 PST
In some cases, pypi serves invalid html. At the moment, this is a pretty narrow exception revolving around the <meta> tag.
Attachments
Patch (2.81 KB, patch)
2021-01-28 16:57 PST, Jonathan Bedard 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jonathan Bedard
Comment 1 2021-01-28 16:52:20 PST
<rdar://problem/73728580>
Jonathan Bedard
Comment 2 2021-01-28 16:57:09 PST
Created attachment 418681 [details] Patch
David Kilzer (:ddkilzer)
Comment 3 2021-01-28 16:59:46 PST
(In reply to Jonathan Bedard from comment #2) > Created attachment 418681 [details] > Patch ERROR: Tools/Scripts/libraries/webkitcorepy/webkitcorepy/autoinstall.py:131: invalid escape sequence '\s' [pep8/W605] [5] Total errors found: 1 in 4 files *shrug*
Jonathan Bedard
Comment 4 2021-01-28 17:00:59 PST
(In reply to David Kilzer (:ddkilzer) from comment #3) > (In reply to Jonathan Bedard from comment #2) > > Created attachment 418681 [details] > > Patch > > ERROR: Tools/Scripts/libraries/webkitcorepy/webkitcorepy/autoinstall.py:131: > invalid escape sequence '\s' [pep8/W605] [5] > Total errors found: 1 in 4 files > > *shrug* Yeah, I don't really understand what it's complaining about. Probably the fact that it should be r'...' instead of b'...' in python 2, but it needs to be b'...' in Python 3, so.....
David Kilzer (:ddkilzer)
Comment 5 2021-01-28 17:02:40 PST
Comment on attachment 418681 [details] Patch r=me
EWS
Comment 6 2021-01-28 17:50:35 PST
Committed r272041: <https://trac.webkit.org/changeset/272041> All reviewed patches have been landed. Closing bug and clearing flags on attachment 418681 [details].
Note You need to log in before you can comment on or make changes to this bug.