[
https://issues.apache.org/jira/browse/SOLR-10943?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
David Smiley closed SOLR-10943.
-------------------------------
Resolution: Invalid
Woops; you're right Yonik; no more escaping needed here.
> SolrJ ClientUtils.escapeQueryChars should escape '$' to avoid Solr macro
> expansion
> ----------------------------------------------------------------------------------
>
> Key: SOLR-10943
> URL: https://issues.apache.org/jira/browse/SOLR-10943
> Project: Solr
> Issue Type: Bug
> Security Level: Public(Default Security Level. Issues are Public)
> Components: SolrJ
> Affects Versions: 5.1
> Reporter: David Smiley
>
> Since Solr 5.1, parameters containing
> {noformat}
> foo${bar}baz
> {noformat} syntax will have parameter substitution performed, and will lead
> to an error if the parameter fails to resolve. ClientUtils.escapeQueryChars
> should escape the dollar sign. Heck, perhaps it should simply escape any
> char that isn't a letter, digit, or whitespace?
> I think this would be good but it admittedly isn't a cure-all for
> unanticipated macro expansions. For example if a client app sends 'q' to be
> processed by edismax, then ClientUtils.escapeQueryChars shouldn't be used
> (user may want quoting, for example; this shouldn't be escaped).
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org
