dimas-b commented on PR #2941: URL: https://github.com/apache/polaris/pull/2941#issuecomment-3469770409
The `quarkus.http.header` approach looks interesting and promising from my POV :slightly_smiling_face: It will also allow downstream builds to control whether the version header is exposed or not with per-deployment granularity (which is sufficient IMHO). However, users of ASF Polaris binaries may not be able to "unset" the default value of this property :thinking: Those users will only be able to override it to something like `quarkus.http.header."Server".value=NotAvailable`. Alternatively, playing with the `path` and/or `method` sub-properties may be needed to exclude this header from response... All in all, this might still be fine for end users, but we'll probably need to document it well in comments. -- 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]
