Title: [233107] trunk/Tools
- Revision
- 233107
- Author
- [email protected]
- Date
- 2018-06-22 16:05:50 -0700 (Fri, 22 Jun 2018)
Log Message
Security EWS: bots fails with exception 'NoneType' object has no attribute 'is_closed'
https://bugs.webkit.org/show_bug.cgi?id=186923
Reviewed by Lucas Forschler.
Attachments downloaded from the status server (http://webkit-queues.webkit.org) do not
have an associated Bug object (i.e. Attachment.bug() is None) and reflect the state they
were in, including flags set, at the time they were uploaded to the status server. We
will add support for fetching up-to-date status info about the attachment and its bug
in <https://bugs.webkit.org/show_bug.cgi?id=186817>. For now, we only check if the
bug associated with the patch we are processing is closed if the attachment has a
non-None Bug object.
* Scripts/webkitpy/tool/bot/earlywarningsystemtask.py:
(EarlyWarningSystemTask.validate):
Modified Paths
Diff
Modified: trunk/Tools/ChangeLog (233106 => 233107)
--- trunk/Tools/ChangeLog 2018-06-22 23:05:47 UTC (rev 233106)
+++ trunk/Tools/ChangeLog 2018-06-22 23:05:50 UTC (rev 233107)
@@ -1,3 +1,21 @@
+2018-06-22 Daniel Bates <[email protected]>
+
+ Security EWS: bots fails with exception 'NoneType' object has no attribute 'is_closed'
+ https://bugs.webkit.org/show_bug.cgi?id=186923
+
+ Reviewed by Lucas Forschler.
+
+ Attachments downloaded from the status server (http://webkit-queues.webkit.org) do not
+ have an associated Bug object (i.e. Attachment.bug() is None) and reflect the state they
+ were in, including flags set, at the time they were uploaded to the status server. We
+ will add support for fetching up-to-date status info about the attachment and its bug
+ in <https://bugs.webkit.org/show_bug.cgi?id=186817>. For now, we only check if the
+ bug associated with the patch we are processing is closed if the attachment has a
+ non-None Bug object.
+
+ * Scripts/webkitpy/tool/bot/earlywarningsystemtask.py:
+ (EarlyWarningSystemTask.validate):
+
2018-06-22 Ross Kirsling <[email protected]>
[Win] ImageDiff should use DLLLauncher
Modified: trunk/Tools/Scripts/webkitpy/tool/bot/earlywarningsystemtask.py (233106 => 233107)
--- trunk/Tools/Scripts/webkitpy/tool/bot/earlywarningsystemtask.py 2018-06-22 23:05:47 UTC (rev 233106)
+++ trunk/Tools/Scripts/webkitpy/tool/bot/earlywarningsystemtask.py 2018-06-22 23:05:50 UTC (rev 233107)
@@ -41,11 +41,15 @@
self._should_build = should_build
def validate(self):
+ # FIXME: Need a way to ask the status server for latest status of a security bug.
+ # Attachments downloaded from the status server do not have an associated bug and
+ # reflect the Bugzilla state at the time they were uploaded to the status server.
+ # See <https://bugs.webkit.org/show_bug.cgi?id=186817>.
self._patch = self._delegate.refetch_patch(self._patch)
if self._patch.is_obsolete():
self.error = "Patch is obsolete."
return False
- if self._patch.bug().is_closed():
+ if self._patch.bug() and self._patch.bug().is_closed():
self.error = "Bug is already closed."
return False
if self._patch.review() == "-":
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
