gianm commented on PR #18404: URL: https://github.com/apache/druid/pull/18404#issuecomment-3189160699
> If we are doing a `34.0.1` release, it would be nice to include this bugfix as well: #18395 > > cc: @cryptoe , @FrankChen021 , @jtuglu1 Let's discuss this on a dev list thread for 34.0.1, it's off topic for the current PR. IMO we should do 34.0.1 since this change breaks a relatively common way of specifying the json content-type. I suppose we didn't notice it in testing because our tests all use the `application/json` form rather than setting any options. > A bug was introduced with this PR where a sql query request going to the router/broker with content type parameters such as `Content-Type: application/json; text/plain` will always fail. I believe this example is invalid, I don’t think requests can have multiple content-types. The error message has an example of a valid content-type: `application/json; charset=UTF-8` means `application/json` with the `charset=UTF-8` option. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
