On Thu, Apr 11, 2013 at 12:56 PM, Maurizio Cucchiara <mcucchi...@apache.org> wrote: > Hi Christian, > according with json spec [1], you should also escape solidus char > (slash for friends :) )
thanks for avoiding me look dumb :-) Anyway, my lib is mainly written for my own use. It seems I have never sent a slash with json, so it didn't pop up. That said I will move on to escapeJavaScript now, which I actually missed. > Anyway, have you already taken a look at LANG-797 [2 Looks good at first glance - what prevents you to commit it? > PS: This discussion should be moved to the Dev List. Indeed. :-) Cheers Christian > > [1] http://goo.gl/EXRqX > [2] https://issues.apache.org/jira/browse/LANG-797 > > On 11 April 2013 12:40, Christian Grobmeier <grobme...@gmail.com> wrote: >> private void encodeString(String string, StringBuilder result, JSON >> annotation) { >> if(string == null) { >> result.append(NULL); >> } else { >> result.append(QUOTE); >> result.append(StringEscapeUtils.escapeJava(string)); >> result.append(QUOTE); >> } >> } > Twitter :http://www.twitter.com/m_cucchiara > G+ :https://plus.google.com/107903711540963855921 > Linkedin :http://www.linkedin.com/in/mauriziocucchiara > VisualizeMe: http://vizualize.me/maurizio.cucchiara?r=maurizio.cucchiara > > Maurizio Cucchiara > > --------------------------------------------------------------------- > To unsubscribe, e-mail: user-unsubscr...@commons.apache.org > For additional commands, e-mail: user-h...@commons.apache.org > -- http://www.grobmeier.de https://www.timeandbill.de --------------------------------------------------------------------- To unsubscribe, e-mail: user-unsubscr...@commons.apache.org For additional commands, e-mail: user-h...@commons.apache.org