Hey Costin, you spotted a really stupid bug, I have inverted CR with LF in the quoting...<br>
well I think you may well be the first one trying to encode CRs and LFs.<br>
<br>
Cooking the (equally stupid) fix now.<br>
<br>
XrdTpc is maintained by Brian. One more good catch!<br>
<br>
Thanks<br>
Fabrizio<br>
<br>
<br>
<br>
On 2020-01-07 15:18, Costin Grigoras wrote:<br>
&gt; Ciao Fabrizio,<br>
&gt; <br>
&gt; The client doesn&#39;t make any difference, the Location header is returned by the server. I think the issue is here:<br>
&gt; <br>
&gt; https://github.com/xrootd/xrootd/blob/master/src/XrdHttp/XrdHttpUtils.cc#L378<br>
&gt; <br>
&gt; |case &#39;\n&#39;: strcpy(r + j, &quot;%0C&quot;); j += 3; break; case &#39;\r&#39;: strcpy(r + j, &quot;%0A&quot;); j += 3; break; |<br>
&gt; <br>
&gt; The correct values (https://en.wikipedia.org/wiki/Newline#Representation) are &quot;%0A&quot; for &#39;\n&#39; and &quot;%0D&quot; (!) for &#39;\r&#39;. Sorry for<br>
&gt; not spotting this before!<br>
&gt; <br>
&gt; I found another, outdated, occurrence of the quote() method in XrdTpcTPC.cc. Maybe that should also be updated as per (or<br>
&gt; consolidated with) the above at this point.<br>
&gt; <br>
&gt; Cheers,<br>
&gt; <br>
&gt; .costin<br>
&gt; <br>
&gt; —<br>
&gt; You are receiving this because you were mentioned.<br>
&gt; Reply to this email directly, view it on GitHub<br>
&gt; &lt;https://github.com/xrootd/xrootd/issues/1060?email_source=notifications&amp;email_token=ABJBUT6S2DBZSC3FUEDEKL3Q4SFLHA5CNFSM4I3HS6F2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEII77WY#issuecomment-571604955&gt;,<br>
&gt; or unsubscribe &lt;https://github.com/notifications/unsubscribe-auth/ABJBUT5L54DKWYRNCFQTSBLQ4SFLHANCNFSM4I3HS6FQ&gt;.<br>
&gt; <br>
<br>


<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/xrootd/xrootd/issues/1060?email_source=notifications&amp;email_token=AA7NRDUERCNQ23MX7C3IXELQ4SJMPA5CNFSM4I3HS6F2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIJDTLQ#issuecomment-571619758">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AA7NRDU4PTXP3DQD4AJZGU3Q4SJMPANCNFSM4I3HS6FQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AA7NRDRAR5XWQV4GK5JBGRTQ4SJMPA5CNFSM4I3HS6F2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIJDTLQ.gif" height="1" width="1" alt="" /></p>
<!cript type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/xrootd/xrootd/issues/1060?email_source=notifications\u0026email_token=AA7NRDUERCNQ23MX7C3IXELQ4SJMPA5CNFSM4I3HS6F2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIJDTLQ#issuecomment-571619758",
"url": "https://github.com/xrootd/xrootd/issues/1060?email_source=notifications\u0026email_token=AA7NRDUERCNQ23MX7C3IXELQ4SJMPA5CNFSM4I3HS6F2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIJDTLQ#issuecomment-571619758",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script><br>
<hr>
<p align="left">
Use REPLY-ALL to reply to list</p>
<p align="center">To unsubscribe from the XROOTD-DEV list, click the following link:<br>
<a href="https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1" target="_blank">https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1</a>
</p>