gerlowskija commented on code in PR #2876:
URL: https://github.com/apache/solr/pull/2876#discussion_r1868027727

##########
solr/core/src/java/org/apache/solr/api/V2HttpCall.java:
##########
@@ -434,7 +434,12 @@ protected void executeCoreRequest(SolrQueryResponse rsp) {
         // SolrCore counter
         core.close();
         core = null;
-        response.getHeaderNames().stream().forEach(name -> 
response.setHeader(name, null));
+        // Skip specific headers
+        // workaround for response.setHeader(name, null)
+        response.getHeaderNames().stream()

Review Comment:
   This looks good for now IMO.
   
   Longer term I'd love to see this code go away in favor of having the v2 API 
stuff get handled by its own servlet, but we're pretty far from that being 
workable for now.  It is a little hacky in the interim though 😦 



-- 
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