https://dev.arvados.org/https://dev.arvados.org/favicon.ico?15576888422018-10-19T13:46:40ZArvadosArvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=679592018-10-19T13:46:40ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li></ul> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=679602018-10-19T13:49:15ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Subject</strong> changed from <i>Java SDK double slash</i> to <i>Java SDK double slash bug with arvados-controller</i></li><li><strong>Description</strong> updated (<a title="View differences" href="/journals/67960/diff?detail_id=64985">diff</a>)</li><li><strong>Status</strong> changed from <i>In Progress</i> to <i>New</i></li></ul> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=679612018-10-19T15:24:57ZTom Morristfmorris@veritasgenetics.com
<ul><li><strong>Subject</strong> changed from <i>Java SDK double slash bug with arvados-controller</i> to <i>[API] Java SDK double slash bug with arvados-controller</i></li></ul><p>Actually, the correct fix is to fix the server so that it isn't breaking existing clients.</p> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=679622018-10-19T15:32:57ZTom Morristfmorris@veritasgenetics.com
<ul><li><strong>Target version</strong> set to <i>2018-11-14 Sprint</i></li></ul> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=683272018-10-31T15:49:48ZTom Cleggtom@curii.com
<ul></ul><p>I wonder why <a class="changeset" title="13497: Avoid misdirecting POST to GET when request has double slash. Arvados-DCO-1.1-Signed-off-..." href="https://dev.arvados.org/projects/arvados/repository/arvados/revisions/f0d62ea064d32b980f723e37972788cbd693c2ff">f0d62ea064d32b980f723e37972788cbd693c2ff</a> didn't fix this. Merged Fri Jul 13 2018.</p> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=683292018-10-31T15:52:04ZTom Cleggtom@curii.com
<ul><li><strong>Related to</strong> <i><a class="issue tracker-6 status-3 priority-4 priority-default closed parent" href="/issues/13497">Idea #13497</a>: [API] Initial "arvados-controller" server that proxies API endpoints to Rails server</i> added</li></ul> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=683312018-10-31T15:52:40ZTom Morristfmorris@veritasgenetics.com
<ul><li><strong>Assigned To</strong> set to <i>Tom Morris</i></li></ul> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=688172018-11-14T16:23:59ZTom Morristfmorris@veritasgenetics.com
<ul><li><strong>Target version</strong> changed from <i>2018-11-14 Sprint</i> to <i>2018-11-28 Sprint</i></li></ul> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=688462018-11-14T21:04:15ZTom Cleggtom@curii.com
<ul></ul><p>Tom Morris wrote:</p>
<blockquote>
<p>Actually, the correct fix is to fix the server so that it isn't breaking existing clients.</p>
</blockquote>
<p>Continuing to tolerate client bugs that were tolerated before should be easy enough here. But the buggy client should still be fixed. API endpoints are "/arvados/...", not "//arvados/...".</p> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=694542018-12-03T22:45:16ZTom Morristfmorris@veritasgenetics.com
<ul><li><strong>Release</strong> set to <i>14</i></li></ul> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=694612018-12-03T22:46:57ZTom Morristfmorris@veritasgenetics.com
<ul><li><strong>Target version</strong> changed from <i>2018-11-28 Sprint</i> to <i>2018-12-12 Sprint</i></li><li><strong>Release</strong> deleted (<del><i>14</i></del>)</li></ul> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=697452018-12-12T16:34:45ZTom Morristfmorris@veritasgenetics.com
<ul><li><strong>Target version</strong> changed from <i>2018-12-12 Sprint</i> to <i>2018-12-21 Sprint</i></li></ul> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=697622018-12-12T17:15:03ZTom Cleggtom@curii.com
<ul></ul><p>Tom Clegg wrote:</p>
<blockquote>
<p>I wonder why <a class="changeset" title="13497: Avoid misdirecting POST to GET when request has double slash. Arvados-DCO-1.1-Signed-off-..." href="https://dev.arvados.org/projects/arvados/repository/arvados/revisions/f0d62ea064d32b980f723e37972788cbd693c2ff">f0d62ea064d32b980f723e37972788cbd693c2ff</a> didn't fix this. Merged Fri Jul 13 2018.</p>
</blockquote>
<p>That "fix path transparently" bit doesn't apply to GET reqs, because following a 301 redirect doesn't change the request method. If a client simply can't handle redirects, even GET requests fail.</p>
<p>I think it's OK to say clients need to be able to follow 301 redirects -- especially clients that aren't paying attention to the paths specified in the discovery doc.</p>
<p>The best fix would be to make Java SDK follow redirects, <em>and</em> fix that broken path to avoid the extra request.</p> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=701532019-01-02T16:13:28ZTom Morristfmorris@veritasgenetics.com
<ul><li><strong>Target version</strong> changed from <i>2018-12-21 Sprint</i> to <i>2019-01-16 Sprint</i></li></ul> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=706972019-01-16T16:20:59ZTom Morristfmorris@veritasgenetics.com
<ul><li><strong>Target version</strong> changed from <i>2019-01-16 Sprint</i> to <i>2019-01-30 Sprint</i></li></ul> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=710012019-01-30T16:10:28ZTom Morristfmorris@veritasgenetics.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li><li><strong>Target version</strong> changed from <i>2019-01-30 Sprint</i> to <i>2019-02-13 Sprint</i></li></ul> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=712982019-02-13T16:06:54ZTom Morristfmorris@veritasgenetics.com
<ul><li><strong>Target version</strong> changed from <i>2019-02-13 Sprint</i> to <i>2019-02-27 Sprint</i></li></ul> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=714682019-02-15T15:01:27ZPeter Amstutzpeter.amstutz@curii.com
<ul></ul><p>LGTM</p> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=717702019-02-27T16:07:52ZTom Morristfmorris@veritasgenetics.com
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Resolved</i></li></ul> Arvados - Bug #14383: [API] Java SDK double slash bug with arvados-controllerhttps://dev.arvados.org/issues/14383?journal_id=719402019-03-01T19:33:55ZTom Morristfmorris@veritasgenetics.com
<ul><li><strong>Release</strong> set to <i>15</i></li></ul>