fluent / fluent-plugin-opensearch / 35186385420

93%

main: 93%

Pull #192

github

out_opensearch: check a host placeholder value by its place

Before: a value could hold a host name and a port wherever the
placeholder was.

* `host myhost-${pipeline_id}` let a record field move the request to
  another domain
* `hosts https://logs-${pipeline_id}.example.com/os` let a value put a
  port in the middle of a name, which `URI()` cannot read
* An empty value dropped the host without an error
* An IPv6 address was rejected everywhere

After: what a value may hold depends on where the operator put the
placeholder.

* A value that continues a name the operator wrote fills in one label,
  so it can add neither a "." nor a port
* A value that stands for the whole host name may hold a name and a port
* An IPv6 address is allowed where the operator wrote the "[]" or the
  scheme that `URI()` needs to read it
* An empty value is rejected everywhere

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

41 of 41 new or added lines in 2 files covered. (100.0%)

1229 of 1321 relevant lines covered (93.04%)

325.22 hits per line