iamsanjay commented on PR #2876:
URL: https://github.com/apache/solr/pull/2876#issuecomment-2507837337

   Recent commit fix the test cases related to V2 api. For now, it's just a 
workaround and I'll discuss with Jason to come up with a better solution. 
Earlier we were setting all the headers null, however the change in library 
does not provide one universal method `removeHeader` but separate methods to 
deal with all the headers. 
   
   Not valid anymore:
   ```
   response.setHeader(name, null)
   ```
   Changed to:
   ```
   response.getHeaderNames().stream()
               .filter(name -> !name.equalsIgnoreCase("Content-Length"))
               .forEach(name -> response.setHeader(name, ""));
           response.setContentLength(-1);
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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

Reply via email to