https://www.redmine.org/https://www.redmine.org/favicon.ico?16793021292015-01-31T14:30:55ZRedmineRedmine - Defect #18922: rdm-mailhandler.rb should catch EOFErrorhttps://www.redmine.org/issues/18922?journal_id=612232015-01-31T14:30:55ZJean-Philippe Langjp_lang@yahoo.fr
<ul></ul><p>Maybe we could even catch IOError which is the superclass of EOFError?</p> Redmine - Defect #18922: rdm-mailhandler.rb should catch EOFErrorhttps://www.redmine.org/issues/18922?journal_id=612272015-02-01T02:07:24ZGo MAEDA
<ul></ul><p>Jean-Philippe Lang wrote:</p>
<blockquote>
<p>Maybe we could even catch IOError which is the superclass of EOFError?</p>
</blockquote>
<p>Indeed. To deal with a variety of situations, catching IOError is better solition.<br />Thank you for pointing it out.</p> Redmine - Defect #18922: rdm-mailhandler.rb should catch EOFErrorhttps://www.redmine.org/issues/18922?journal_id=612282015-02-01T02:18:40ZGo MAEDA
<ul><li><strong>File</strong> <a href="/attachments/13065">catch-IOError.diff</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/13065/catch-IOError.diff">catch-IOError.diff</a> added</li></ul><p>Here is a new patch. Catches IOError instead of EOFError.</p> Redmine - Defect #18922: rdm-mailhandler.rb should catch EOFErrorhttps://www.redmine.org/issues/18922?journal_id=613482015-02-07T07:37:56ZJean-Philippe Langjp_lang@yahoo.fr
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Resolved</i></li><li><strong>Assignee</strong> set to <i>Jean-Philippe Lang</i></li><li><strong>Target version</strong> set to <i>2.6.2</i></li><li><strong>Resolution</strong> set to <i>Fixed</i></li></ul><p>Patch committed in <a class="changeset" title="rdm-mailhandler.rb should catch EOFError (#18922). Patch by Go MAEDA." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/13961">r13961</a>, thanks.</p> Redmine - Defect #18922: rdm-mailhandler.rb should catch EOFErrorhttps://www.redmine.org/issues/18922?journal_id=615852015-02-18T11:35:09ZJean-Philippe Langjp_lang@yahoo.fr
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Closed</i></li></ul>