suvodeep-pyne opened a new pull request, #16694: URL: https://github.com/apache/pinot/pull/16694
## Summary - Transform captureRequestHeaders configuration from boolean to comma-separated list of specific headers to capture - Implement case-insensitive header matching per HTTP RFC 7230 - Add comprehensive test coverage for edge cases and various string formats - Refactor header filtering logic to reuse existing toMap function with allow list parameter ## Changes - **AuditConfig**: Change `_captureRequestHeaders` from `boolean` to `String` - **AuditRequestProcessor**: Add `parseAllowedHeaders()` utility and enhance `toMap()` with filtering - **Tests**: Add comprehensive test coverage for configuration parsing, case sensitivity, value handling, and end-to-end flow ## Test Coverage - Configuration parsing edge cases (empty, whitespace, malformed comma separation) - Case-insensitive HTTP header matching - Single vs multiple header values handling - Complete request processing integration tests ## Breaking Changes None - this is an unlaunched feature in active development. -- 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]
