New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport 2.x] [FEATURE] Broaden SecureSettingsFactory to include http transports (#12907) #12965

Merged

peternied merged 1 commit into opensearch-project:2.x from reta:backport/backport-12907-to-2.x

Mar 28, 2024

Merged

[Backport 2.x] [FEATURE] Broaden SecureSettingsFactory to include http transports (#12907) #12965

[FEATURE] Broaden SecureSettingsFactory to include http transports (#…

Codecov / codecov/patch succeeded Mar 28, 2024 in 0s

90.00% of diff hit (target 71.28%)

View this Pull Request on Codecov

90.00% of diff hit (target 71.28%)

Annotations

Check warning on line 129 in modules/transport-netty4/src/main/java/org/opensearch/http/netty4/ssl/SecureNetty4HttpServerTransport.java

codecov / codecov/patch

modules/transport-netty4/src/main/java/org/opensearch/http/netty4/ssl/SecureNetty4HttpServerTransport.java#L129

Added line #L129 was not covered by tests

Check warning on line 178 in modules/transport-netty4/src/main/java/org/opensearch/http/netty4/ssl/SecureNetty4HttpServerTransport.java

codecov / codecov/patch

modules/transport-netty4/src/main/java/org/opensearch/http/netty4/ssl/SecureNetty4HttpServerTransport.java#L178

Added line #L178 was not covered by tests

Check warning on line 36 in server/src/main/java/org/opensearch/plugins/SecureTransportSettingsProvider.java

codecov / codecov/patch

server/src/main/java/org/opensearch/plugins/SecureTransportSettingsProvider.java#L36

Added line #L36 was not covered by tests

View more details on Codecov