<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Aug 30, 2016 at 6:01 PM Tom Zeller <<a href="mailto:tzeller@dragonacea.biz">tzeller@dragonacea.biz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I don't know how it all works, but the Spring UriComponentsBuilder<br>
doesn't encode query params, AFAIK.<br></blockquote><div><br></div><div>Of course it's more complicated than that. Careful reading of RFC 3986 indicates that the Spring component is properly encoding the ticket as a URI. Specifically, "+", "/", and "=" are allowed in the query portion of a URI:</div><div><br></div><div>query         = *( pchar / "/" / "?" )<br></div><div>pchar         = unreserved / pct-encoded / sub-delims / ":" / "@"<br></div><div><div>sub-delims    = "!" / "$" / "&" / "'" / "(" / ")" <span style="line-height:1.5">"*" / "+" / "," / ";" / "="</span></div></div><div><span style="line-height:1.5"><br></span></div><div>However, many libraries don't expect those characters in the query part and expect them to be percent encoded. Explicitly encoding the ticket is the right thing to do; doesn't hurt any RFC-compliant libs and helps the rest. I intend to commit a fix and unit test coverage today.</div><div><br></div><div>M<a href="mailto:dev-unsubscribe@shibboleth.net" target="_blank"></a><br>
</div><div><br></div></div></div>