221111 2021-01-28 16:51:08 -0800 [webkitcrepy] Handle case where pypi serves invalid html 2021-01-29 16:22:02 -0800 1 1 1 Unclassified WebKit Tools / Tests WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=221162 InRadar P2 Normal --- 1 jbedard jbedard aakash_jain ddkilzer dewei_zhu slewis webkit-bug-importer oldest_to_newest 1723547 0 jbedard 2021-01-28 16:51:08 -0800 In some cases, pypi serves invalid html. At the moment, this is a pretty narrow exception revolving around the <meta> tag. 1723548 1 jbedard 2021-01-28 16:52:20 -0800 <rdar://problem/73728580> 1723549 2 418681 jbedard 2021-01-28 16:57:09 -0800 Created attachment 418681 Patch 1723550 3 ddkilzer 2021-01-28 16:59:46 -0800 (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* 1723551 4 jbedard 2021-01-28 17:00:59 -0800 (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..... 1723552 5 418681 ddkilzer 2021-01-28 17:02:40 -0800 Comment on attachment 418681 Patch r=me 1723565 6 ews-feeder 2021-01-28 17:50:35 -0800 Committed r272041: <https://trac.webkit.org/changeset/272041> All reviewed patches have been landed. Closing bug and clearing flags on attachment 418681. 418681 2021-01-28 16:57:09 -0800 2021-01-28 17:50:36 -0800 Patch bug-221111-20210128165708.patch text/plain 2879 jbedard SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl dmlzaW9uIDI3MjA0MCkKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz ICsxLDE2IEBACisyMDIxLTAxLTI4ICBKb25hdGhhbiBCZWRhcmQgIDxqYmVkYXJkQGFwcGxlLmNv bT4KKworICAgICAgICBbd2Via2l0Y3JlcHldIEhhbmRsZSBjYXNlIHdoZXJlIHB5cGkgc2VydmVz IGludmFsaWQgaHRtbAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5j Z2k/aWQ9MjIxMTExCisgICAgICAgIDxyZGFyOi8vcHJvYmxlbS83MzcyODU4MD4KKworICAgICAg ICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIFNjcmlwdHMvbGlicmFy aWVzL3dlYmtpdGNvcmVweS9zZXR1cC5weTogQnVtcCB2ZXJzaW9uIG51bWJlci4KKyAgICAgICAg KiBTY3JpcHRzL2xpYnJhcmllcy93ZWJraXRjb3JlcHkvd2Via2l0Y29yZXB5L19faW5pdF9fLnB5 OiBEaXR0by4KKyAgICAgICAgKiBTY3JpcHRzL2xpYnJhcmllcy93ZWJraXRjb3JlcHkvd2Via2l0 Y29yZXB5L2F1dG9pbnN0YWxsLnB5OgorICAgICAgICAoUGFja2FnZS5hcmNoaXZlcyk6IFJlbW92 ZSBwb3NzaWJseSBpbnZhbGlkIDxtZXRhPiB0YWcgZnJvbSBodG1sLgorCiAyMDIxLTAxLTI4ICBB YWthc2ggSmFpbiAgPGFha2FzaF9qYWluQGFwcGxlLmNvbT4KIAogICAgICAgICBbYnVpbGQud2Vi a2l0Lm9yZ10gUmVtb3ZlIGRlYWQgY29kZSBmb3IgVGVzdFdlYktpdDFMZWFrc0ZhY3RvcnkKSW5k ZXg6IFRvb2xzL1NjcmlwdHMvbGlicmFyaWVzL3dlYmtpdGNvcmVweS9zZXR1cC5weQo9PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09Ci0tLSBUb29scy9TY3JpcHRzL2xpYnJhcmllcy93ZWJraXRjb3JlcHkvc2V0dXAucHkJKHJl dmlzaW9uIDI3MjAzOCkKKysrIFRvb2xzL1NjcmlwdHMvbGlicmFyaWVzL3dlYmtpdGNvcmVweS9z ZXR1cC5weQkod29ya2luZyBjb3B5KQpAQCAtMzAsNyArMzAsNyBAQCBkZWYgcmVhZG1lKCk6CiAK IHNldHVwKAogICAgIG5hbWU9J3dlYmtpdGNvcmVweScsCi0gICAgdmVyc2lvbj0nMC41LjAnLAor ICAgIHZlcnNpb249JzAuNS4xJywKICAgICBkZXNjcmlwdGlvbj0nTGlicmFyeSBjb250YWluaW5n IHZhcmlvdXMgUHl0aG9uIHN1cHBvcnQgY2xhc3NlcyBhbmQgZnVuY3Rpb25zLicsCiAgICAgbG9u Z19kZXNjcmlwdGlvbj1yZWFkbWUoKSwKICAgICBjbGFzc2lmaWVycz1bCkluZGV4OiBUb29scy9T Y3JpcHRzL2xpYnJhcmllcy93ZWJraXRjb3JlcHkvd2Via2l0Y29yZXB5L19faW5pdF9fLnB5Cj09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT0KLS0tIFRvb2xzL1NjcmlwdHMvbGlicmFyaWVzL3dlYmtpdGNvcmVweS93ZWJraXRj b3JlcHkvX19pbml0X18ucHkJKHJldmlzaW9uIDI3MjAzOCkKKysrIFRvb2xzL1NjcmlwdHMvbGli cmFyaWVzL3dlYmtpdGNvcmVweS93ZWJraXRjb3JlcHkvX19pbml0X18ucHkJKHdvcmtpbmcgY29w eSkKQEAgLTM2LDcgKzM2LDcgQEAgZnJvbSB3ZWJraXRjb3JlcHkuc3VicHJvY2Vzc191dGlscyBp bXBvcgogZnJvbSB3ZWJraXRjb3JlcHkub3V0cHV0X2NhcHR1cmUgaW1wb3J0IExvZ2dlckNhcHR1 cmUsIE91dHB1dENhcHR1cmUsIE91dHB1dER1cGxpY2F0ZQogZnJvbSB3ZWJraXRjb3JlcHkudGFz a19wb29sIGltcG9ydCBUYXNrUG9vbAogCi12ZXJzaW9uID0gVmVyc2lvbigwLCA1LCAwKQordmVy c2lvbiA9IFZlcnNpb24oMCwgNSwgMSkKIAogZnJvbSB3ZWJraXRjb3JlcHkuYXV0b2luc3RhbGwg aW1wb3J0IFBhY2thZ2UsIEF1dG9JbnN0YWxsCiBpZiBzeXMudmVyc2lvbl9pbmZvID4gKDMsIDAp OgpJbmRleDogVG9vbHMvU2NyaXB0cy9saWJyYXJpZXMvd2Via2l0Y29yZXB5L3dlYmtpdGNvcmVw eS9hdXRvaW5zdGFsbC5weQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9TY3JpcHRzL2xpYnJhcmllcy93 ZWJraXRjb3JlcHkvd2Via2l0Y29yZXB5L2F1dG9pbnN0YWxsLnB5CShyZXZpc2lvbiAyNzIwMzgp CisrKyBUb29scy9TY3JpcHRzL2xpYnJhcmllcy93ZWJraXRjb3JlcHkvd2Via2l0Y29yZXB5L2F1 dG9pbnN0YWxsLnB5CSh3b3JraW5nIGNvcHkpCkBAIC0xMjYsNyArMTI2LDExIEBAIGNsYXNzIFBh Y2thZ2Uob2JqZWN0KToKICAgICAgICAgICAgIGlmIHJlc3BvbnNlLmNvZGUgIT0gMjAwOgogICAg ICAgICAgICAgICAgIHJhaXNlIFZhbHVlRXJyb3IoJ1RoZSBwYWNrYWdlIHt9IHdhcyBub3QgZm91 bmQgb24ge30nLmZvcm1hdChzZWxmLnB5cGlfbmFtZSwgQXV0b0luc3RhbGwuaW5kZXgpKQogCi0g ICAgICAgICAgICBwYWdlID0gbWluaWRvbS5wYXJzZVN0cmluZyhyZXNwb25zZS5yZWFkKCkpCisg ICAgICAgICAgICAjIEluIHNvbWUgY2FzZXMsIHB5cGkgc2VydmVzIGludmFsaWQgaHRtbAorICAg ICAgICAgICAgY29udGVudCA9IHJlc3BvbnNlLnJlYWQoKQorICAgICAgICAgICAgY29udGVudCA9 IHJlLnN1YihiJ1xzKzxtZXRhIFteXG5dKz5cbicsIGInJywgY29udGVudCkKKworICAgICAgICAg ICAgcGFnZSA9IG1pbmlkb20ucGFyc2VTdHJpbmcoY29udGVudCkKICAgICAgICAgICAgIGNhY2hl ZF90YWdzID0gTm9uZQogCiAgICAgICAgICAgICBmb3IgZWxlbWVudCBpbiByZXZlcnNlZChwYWdl LmdldEVsZW1lbnRzQnlUYWdOYW1lKCJhIikpOgo=