[jira] [Commented] (SOLR-17263) HttpJdkSolrClient doesn't encode curly braces etc
[ https://issues.apache.org/jira/browse/SOLR-17263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17847092#comment-17847092 ] Andy Webb commented on SOLR-17263: -- Thanks [~dsmiley]! I'm happy with all the above, would say that's all we need to merge in for 9.6.1 here. https://github.com/apache/solr/pull/2454 could be included too subject to review, but it's just a refactoring so it can wait. > HttpJdkSolrClient doesn't encode curly braces etc > - > > Key: SOLR-17263 > URL: https://issues.apache.org/jira/browse/SOLR-17263 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) > Components: SolrJ >Affects Versions: 9.6.0 >Reporter: Andy Webb >Priority: Major > Fix For: 9.6.1 > > Time Spent: 2.5h > Remaining Estimate: 0h > > Ref > https://issues.apache.org/jira/browse/SOLR-599?focusedCommentId=17842429#comment-17842429 > - {{HttpJdkSolrClient}} should use {{{}SolrParams{}}}' {{toQueryString()}} > method when constructing URLs to that all URL-unsafe characters are encoded. > It's implicitly using the {{toString()}} method currently which is intended > for logging etc purposes. > Attempting to use alternate query parsers in requests as shown below will > currently fail as the curly braces aren't encoded. > {noformat} > myquery.set("fq", "{!terms f=myfield}value1,value2"); {noformat} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
[jira] [Commented] (SOLR-17263) HttpJdkSolrClient doesn't encode curly braces etc
[ https://issues.apache.org/jira/browse/SOLR-17263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17846638#comment-17846638 ] David Smiley commented on SOLR-17263: - I'm a bit confused by all these PRs but I merged the first one to main, branch_9x, and branch_9_6 as the comments above show. Maybe someone else can help out with the others. > HttpJdkSolrClient doesn't encode curly braces etc > - > > Key: SOLR-17263 > URL: https://issues.apache.org/jira/browse/SOLR-17263 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) > Components: SolrJ >Affects Versions: 9.6.0 >Reporter: Andy Webb >Priority: Major > Fix For: 9.6.1 > > Time Spent: 2h > Remaining Estimate: 0h > > Ref > https://issues.apache.org/jira/browse/SOLR-599?focusedCommentId=17842429#comment-17842429 > - {{HttpJdkSolrClient}} should use {{{}SolrParams{}}}' {{toQueryString()}} > method when constructing URLs to that all URL-unsafe characters are encoded. > It's implicitly using the {{toString()}} method currently which is intended > for logging etc purposes. > Attempting to use alternate query parsers in requests as shown below will > currently fail as the curly braces aren't encoded. > {noformat} > myquery.set("fq", "{!terms f=myfield}value1,value2"); {noformat} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
[jira] [Commented] (SOLR-17263) HttpJdkSolrClient doesn't encode curly braces etc
[ https://issues.apache.org/jira/browse/SOLR-17263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17846634#comment-17846634 ] ASF subversion and git services commented on SOLR-17263: Commit aafdb1be43c2173a1ef757b858d5027e83bd2d89 in solr's branch refs/heads/branch_9_6 from Andy Webb [ https://gitbox.apache.org/repos/asf?p=solr.git;h=aafdb1be43c ] SOLR-17263: HttpJdkSolrClient doesn't encode curly braces etc (#2433) (cherry picked from commit 4c439d07ca35e7e80c5406c103e885307f0b64b9) > HttpJdkSolrClient doesn't encode curly braces etc > - > > Key: SOLR-17263 > URL: https://issues.apache.org/jira/browse/SOLR-17263 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) > Components: SolrJ >Affects Versions: 9.6.0 >Reporter: Andy Webb >Priority: Major > Fix For: 9.6.1 > > Time Spent: 2h > Remaining Estimate: 0h > > Ref > https://issues.apache.org/jira/browse/SOLR-599?focusedCommentId=17842429#comment-17842429 > - {{HttpJdkSolrClient}} should use {{{}SolrParams{}}}' {{toQueryString()}} > method when constructing URLs to that all URL-unsafe characters are encoded. > It's implicitly using the {{toString()}} method currently which is intended > for logging etc purposes. > Attempting to use alternate query parsers in requests as shown below will > currently fail as the curly braces aren't encoded. > {noformat} > myquery.set("fq", "{!terms f=myfield}value1,value2"); {noformat} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
[jira] [Commented] (SOLR-17263) HttpJdkSolrClient doesn't encode curly braces etc
[ https://issues.apache.org/jira/browse/SOLR-17263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17846620#comment-17846620 ] ASF subversion and git services commented on SOLR-17263: Commit be9fcf3d696b2ac484f128a58561c2cf942eaf5b in solr's branch refs/heads/branch_9x from Andy Webb [ https://gitbox.apache.org/repos/asf?p=solr.git;h=be9fcf3d696 ] SOLR-17263: HttpJdkSolrClient doesn't encode curly braces etc (#2433) (cherry picked from commit 4c439d07ca35e7e80c5406c103e885307f0b64b9) > HttpJdkSolrClient doesn't encode curly braces etc > - > > Key: SOLR-17263 > URL: https://issues.apache.org/jira/browse/SOLR-17263 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) > Components: SolrJ >Affects Versions: 9.6.0 >Reporter: Andy Webb >Priority: Major > Fix For: 9.6.1 > > Time Spent: 2h > Remaining Estimate: 0h > > Ref > https://issues.apache.org/jira/browse/SOLR-599?focusedCommentId=17842429#comment-17842429 > - {{HttpJdkSolrClient}} should use {{{}SolrParams{}}}' {{toQueryString()}} > method when constructing URLs to that all URL-unsafe characters are encoded. > It's implicitly using the {{toString()}} method currently which is intended > for logging etc purposes. > Attempting to use alternate query parsers in requests as shown below will > currently fail as the curly braces aren't encoded. > {noformat} > myquery.set("fq", "{!terms f=myfield}value1,value2"); {noformat} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
[jira] [Commented] (SOLR-17263) HttpJdkSolrClient doesn't encode curly braces etc
[ https://issues.apache.org/jira/browse/SOLR-17263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17846619#comment-17846619 ] ASF subversion and git services commented on SOLR-17263: Commit 4c439d07ca35e7e80c5406c103e885307f0b64b9 in solr's branch refs/heads/main from Andy Webb [ https://gitbox.apache.org/repos/asf?p=solr.git;h=4c439d07ca3 ] SOLR-17263: HttpJdkSolrClient doesn't encode curly braces etc (#2433) > HttpJdkSolrClient doesn't encode curly braces etc > - > > Key: SOLR-17263 > URL: https://issues.apache.org/jira/browse/SOLR-17263 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) > Components: SolrJ >Affects Versions: 9.6.0 >Reporter: Andy Webb >Priority: Major > Fix For: 9.6.1 > > Time Spent: 2h > Remaining Estimate: 0h > > Ref > https://issues.apache.org/jira/browse/SOLR-599?focusedCommentId=17842429#comment-17842429 > - {{HttpJdkSolrClient}} should use {{{}SolrParams{}}}' {{toQueryString()}} > method when constructing URLs to that all URL-unsafe characters are encoded. > It's implicitly using the {{toString()}} method currently which is intended > for logging etc purposes. > Attempting to use alternate query parsers in requests as shown below will > currently fail as the curly braces aren't encoded. > {noformat} > myquery.set("fq", "{!terms f=myfield}value1,value2"); {noformat} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
[jira] [Commented] (SOLR-17263) HttpJdkSolrClient doesn't encode curly braces etc
[ https://issues.apache.org/jira/browse/SOLR-17263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17846047#comment-17846047 ] Eric Pugh commented on SOLR-17263: -- I *think* that if we know we are doing a 9.6.1, then you could just do it there under them all? Otherwise, what James said! > HttpJdkSolrClient doesn't encode curly braces etc > - > > Key: SOLR-17263 > URL: https://issues.apache.org/jira/browse/SOLR-17263 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) > Components: SolrJ >Affects Versions: 9.6.0 >Reporter: Andy Webb >Priority: Major > Fix For: 9.6.1 > > Time Spent: 1h 50m > Remaining Estimate: 0h > > Ref > https://issues.apache.org/jira/browse/SOLR-599?focusedCommentId=17842429#comment-17842429 > - {{HttpJdkSolrClient}} should use {{{}SolrParams{}}}' {{toQueryString()}} > method when constructing URLs to that all URL-unsafe characters are encoded. > It's implicitly using the {{toString()}} method currently which is intended > for logging etc purposes. > Attempting to use alternate query parsers in requests as shown below will > currently fail as the curly braces aren't encoded. > {noformat} > myquery.set("fq", "{!terms f=myfield}value1,value2"); {noformat} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
[jira] [Commented] (SOLR-17263) HttpJdkSolrClient doesn't encode curly braces etc
[ https://issues.apache.org/jira/browse/SOLR-17263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17846037#comment-17846037 ] James Dyer commented on SOLR-17263: --- [~andywebb1975] I think you'd be safe following the pattern with SOLR-17261 and add your commit three places: *main*, *branch_9x* and *branch_9_6*. The CHANGES.txt entry would go under 9_7_0 in the two branches, and under 9_6_1 in *branch_9_6*. Those of you who commit more than I do may have better advice than this! > HttpJdkSolrClient doesn't encode curly braces etc > - > > Key: SOLR-17263 > URL: https://issues.apache.org/jira/browse/SOLR-17263 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) > Components: SolrJ >Affects Versions: 9.6.0 >Reporter: Andy Webb >Priority: Major > Fix For: 9.6.1 > > Time Spent: 1h 50m > Remaining Estimate: 0h > > Ref > https://issues.apache.org/jira/browse/SOLR-599?focusedCommentId=17842429#comment-17842429 > - {{HttpJdkSolrClient}} should use {{{}SolrParams{}}}' {{toQueryString()}} > method when constructing URLs to that all URL-unsafe characters are encoded. > It's implicitly using the {{toString()}} method currently which is intended > for logging etc purposes. > Attempting to use alternate query parsers in requests as shown below will > currently fail as the curly braces aren't encoded. > {noformat} > myquery.set("fq", "{!terms f=myfield}value1,value2"); {noformat} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
[jira] [Commented] (SOLR-17263) HttpJdkSolrClient doesn't encode curly braces etc
[ https://issues.apache.org/jira/browse/SOLR-17263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17845630#comment-17845630 ] Andy Webb commented on SOLR-17263: -- [https://github.com/apache/solr/pull/2433] has a fix for this. I'm a relatively new committer and am not not sure how to proceed re merging it - should I add this to both {{branch_9_6}} and {{main}}, and add the {{CHANGES.txt}} line under different versions in those branches like those for SOLR-17261? Subject to review(s) I'd also like to bring in https://github.com/apache/solr/pull/2454 but don't know if that should be in 9.6.1 too, or wait for 9.7.0. > HttpJdkSolrClient doesn't encode curly braces etc > - > > Key: SOLR-17263 > URL: https://issues.apache.org/jira/browse/SOLR-17263 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) > Components: SolrJ >Affects Versions: 9.6.0 >Reporter: Andy Webb >Priority: Major > Fix For: 9.6.1 > > Time Spent: 1h 50m > Remaining Estimate: 0h > > Ref > https://issues.apache.org/jira/browse/SOLR-599?focusedCommentId=17842429#comment-17842429 > - {{HttpJdkSolrClient}} should use {{{}SolrParams{}}}' {{toQueryString()}} > method when constructing URLs to that all URL-unsafe characters are encoded. > It's implicitly using the {{toString()}} method currently which is intended > for logging etc purposes. > Attempting to use alternate query parsers in requests as shown below will > currently fail as the curly braces aren't encoded. > {noformat} > myquery.set("fq", "{!terms f=myfield}value1,value2"); {noformat} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
[jira] [Commented] (SOLR-17263) HttpJdkSolrClient doesn't encode curly braces etc
[ https://issues.apache.org/jira/browse/SOLR-17263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17842725#comment-17842725 ] Andy Webb commented on SOLR-17263: -- Thanks James, and no worries! I want to understand the code better, try to spin this up and exercise it locally before I review the PR further. I'll see what I can do in the next few days. Andy > HttpJdkSolrClient doesn't encode curly braces etc > - > > Key: SOLR-17263 > URL: https://issues.apache.org/jira/browse/SOLR-17263 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) > Components: SolrJ >Affects Versions: 9.6.0 >Reporter: Andy Webb >Priority: Major > Time Spent: 20m > Remaining Estimate: 0h > > Ref > https://issues.apache.org/jira/browse/SOLR-599?focusedCommentId=17842429#comment-17842429 > - {{HttpJdkSolrClient}} should use {{{}SolrParams{}}}' {{toQueryString()}} > method when constructing URLs to that all URL-unsafe characters are encoded. > It's implicitly using the {{toString()}} method currently which is intended > for logging etc purposes. > Attempting to use alternate query parsers in requests as shown below will > currently fail as the curly braces aren't encoded. > {noformat} > myquery.set("fq", "{!terms f=myfield}value1,value2"); {noformat} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
[jira] [Commented] (SOLR-17263) HttpJdkSolrClient doesn't encode curly braces etc
[ https://issues.apache.org/jira/browse/SOLR-17263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17842689#comment-17842689 ] James Dyer commented on SOLR-17263: --- [~andywebb1975] Thank you for being an early adopter of this new client, for reporting the bug and for the PR with the fix. Sorry for the dueling PR's, but can you review my follow-up PR to yours, [2435|https://github.com/apache/solr/pull/2435]? I added a unit test and found sure enough that this affects the Jdk client but the existing Jetty Client handles the parameters properly. The test indicates only GET is affected by this bug but POST and PUT requests are OK. In the follow-up PR I modified your fix to be only for GET. > HttpJdkSolrClient doesn't encode curly braces etc > - > > Key: SOLR-17263 > URL: https://issues.apache.org/jira/browse/SOLR-17263 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) > Components: SolrJ >Affects Versions: 9.6.0 >Reporter: Andy Webb >Priority: Major > Time Spent: 20m > Remaining Estimate: 0h > > Ref > https://issues.apache.org/jira/browse/SOLR-599?focusedCommentId=17842429#comment-17842429 > - {{HttpJdkSolrClient}} should use {{{}SolrParams{}}}' {{toQueryString()}} > method when constructing URLs to that all URL-unsafe characters are encoded. > It's implicitly using the {{toString()}} method currently which is intended > for logging etc purposes. > Attempting to use alternate query parsers in requests as shown below will > currently fail as the curly braces aren't encoded. > {noformat} > myquery.set("fq", "{!terms f=myfield}value1,value2"); {noformat} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org