<table><tr><td style="">leinir updated this revision to Diff 54928.<br />leinir retitled this revision from "Accept HTTP status 103 as benign" to "Accept any HTTP status between 100 and 199 as benign".<br />leinir edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-krvysl2dlhluurl/">(Show Details)</a><br />leinir added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D20077">View Revision</a></tr></table><br /><div><div><p>Just accept any informational status as benign, rather than the specifically picked ones.</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">In December 2017, a new status code <span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">(103) </span>was added to the HTTP standard, and as one prominent OCS provider now uses this status code, Attica needs to be able to handle it more gracefully than just reporting an unknown OCS error (which is reported to the user, but has no actual effect on things).<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> As it is a benign status, we simply accept it and forward it to the consumer as being not an error.</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">So</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Specifically</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">as it is a benign</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">we do this by changing from accepting only status 100 as an informational</span> status, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">we simply</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">to</span> accept<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> it and forward it to the consumer as being not an error</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">ing any informational status (since any actual errors are reported through other status codes)</span>.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R235 Attica</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D20077?vs=54925&id=54928">https://phabricator.kde.org/D20077?vs=54925&id=54928</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20077">https://phabricator.kde.org/D20077</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/atticabasejob.cpp</div></div></div><br /><div><strong>To: </strong>leinir, KNewStuff, Frameworks<br /><strong>Cc: </strong>apol, kde-frameworks-devel, KNewStuff, michaelh, ZrenBot, ngraham, bruns<br /></div>