[ 
https://issues.apache.org/jira/browse/SOLR-14387?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mike Drob resolved SOLR-14387.
------------------------------
    Fix Version/s: master (9.0)
         Assignee: Mike Drob
       Resolution: Fixed

This is a good catch and a good fix! Thanks for the patch, I've pushed it to 
master. I'm a little bit concerned that folks might be improperly relying on 
the old behavior to get multiple ids thinking that it is a convenient shortcut, 
so I'm hesitant to put this in branch_8x. Let me know if you disagree!

> SolrClient.getById() does not escape parameter separators within ids
> --------------------------------------------------------------------
>
>                 Key: SOLR-14387
>                 URL: https://issues.apache.org/jira/browse/SOLR-14387
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: SolrJ
>    Affects Versions: 8.5
>            Reporter: Markus Schuch
>            Assignee: Mike Drob
>            Priority: Major
>             Fix For: master (9.0)
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Having a solr document with a comma in its id (e.g. "A,B"), 
> {{SolrClient.getById()}} is not able to retrieve this document, because it 
> queries {{/get?ids=A,B}} instead of {{/get?ids=A\,B}}.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to