fluent / fluent-plugin-opensearch / 35062263158

93%

main: 93%

Build Type

Pull #192

github

Committed by kenhys

Commit Message

out_opensearch: limit what a host placeholder can expand to

Before: A placeholder in `host`/`hosts` was expanded and passed to the
connection options as is. So a tag or a record field could add a host
with ",", userinfo with "@" or a path with "/". The configured user,
password and custom_headers were then sent to that host.

After: Each placeholder value is checked before the whole setting is
expanded, so a value can only fill in a host name and a port. The rest
of the setting, such as the "," between hosts or a scheme, userinfo
and path the operator wrote, is left as it is. A placeholder used
together with credentials logs a warning at startup.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>

Pull Request Pull Request #192: out_opensearch: limit what a host placeholder can expand to

Coverage Stats

49 of 50 new or added lines in 2 files covered. (98.0%)

23 existing lines in 1 file now uncovered.

1213 of 1305 relevant lines covered (92.95%)

318.46 hits per line