<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Aug 30, 2016 at 11:57 AM 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">Deal is, calling HttpServletRequest.getParameter() returns the latter (the wrong one with the “ “). What I need, I think, is a version of java-support’s URISupport.parseQueryString() which does not URL decode query string params, but I thought I’d ask first.<br></blockquote><div><br></div><div>In short, the ticket string should be URL encoded so that decoding it produces the original ticket ID. I will review tomorrow to determine whether I got sloppy and missed an encoding call. I know that the servlet framework does some encoding automatically, but I'll have to review to say for sure what happens under what circumstances. In any case I'll try to add some test coverage to ensure it's properly encoded.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The issue came up, I think, because the EncodingTicketService introduced characters into the ticket via the DataSealer that my horrible CAS “client” hadn’t seen before.<br></blockquote><div><br></div><div><span style="line-height:1.5">I _knew_ non-safe characters was going to cause problems, but I have to admit I didn't do any careful analysis on exactly what those problems were.</span></div><div><span style="line-height:1.5"><br></span></div><div><span style="line-height:1.5">M</span></div><div><span style="line-height:1.5"><br></span></div></div></div>