Mark Sapiro pushed to branch master at GNU Mailman / Mailman Core


Commits:
5dcbf375 by Mark Sapiro at 2023-02-07T17:38:35-08:00
Don't log StopIteration in workflow.

- - - - -


2 changed files:

- src/mailman/app/workflow.py
- src/mailman/docs/NEWS.rst


Changes:

=====================================
src/mailman/app/workflow.py
=====================================
@@ -67,7 +67,8 @@ class Workflow:
         try:
             name, step = self._pop()
             return step()
-        except IndexError:
+        except (IndexError, StopIteration):
+            # Catch StopIteration so it's not logged. See #1059.
             raise StopIteration
         except:                                     # noqa: E722
             log.exception('deque: {}'.format(COMMASPACE.join(self._next)))


=====================================
src/mailman/docs/NEWS.rst
=====================================
@@ -34,6 +34,8 @@ Bugs fixed
   ``use_ipython: yes``.  (Closes #1062)
 * Bogus Message-ID headers that have been observed in the wild are now fixed.
   (Closes #1065)
+* Raising StopIteration in a workflow process no longer creates a mailman.error
+  log entry.  (Closes #1059)
 
 .. _news-3.3.8:
 



View it on GitLab: 
https://gitlab.com/mailman/mailman/-/commit/5dcbf375cc07e6017adacd5ff7c03e8434bc07ff

-- 
View it on GitLab: 
https://gitlab.com/mailman/mailman/-/commit/5dcbf375cc07e6017adacd5ff7c03e8434bc07ff
You're receiving this email because of your account on gitlab.com.


_______________________________________________
Mailman-checkins mailing list -- mailman-checkins@python.org
To unsubscribe send an email to mailman-checkins-le...@python.org
https://mail.python.org/mailman3/lists/mailman-checkins.python.org/
Member address: arch...@jab.org

Reply via email to