Skip to content
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

Improves connection pooling support for AWSSigV4 clients in data sources. #6135

Merged
merged 7 commits into from
Mar 15, 2024

Revert client pool changes from authentication method

18af6c9
Select commit
Loading
Failed to load commit list.
Merged

Improves connection pooling support for AWSSigV4 clients in data sources. #6135

Revert client pool changes from authentication method
18af6c9
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Mar 15, 2024 in 0s

77.68% of diff hit (target 67.22%)

View this Pull Request on Codecov

77.68% of diff hit (target 67.22%)

Annotations

Check warning on line 27 in src/plugins/data_source/server/auth_registry/authentication_methods_registry.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/plugins/data_source/server/auth_registry/authentication_methods_registry.ts#L27 

Added line #L27 was not covered by tests

Check warning on line 60 in src/plugins/data_source/server/legacy/http_aws_es/connector.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/plugins/data_source/server/legacy/http_aws_es/connector.js#L59-L60 

Added lines #L59 - L60 were not covered by tests

Check warning on line 64 in src/plugins/data_source/server/legacy/http_aws_es/connector.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/plugins/data_source/server/legacy/http_aws_es/connector.js#L64 

Added line #L64 was not covered by tests

Check warning on line 91 in src/plugins/data_source/server/legacy/http_aws_es/connector.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/plugins/data_source/server/legacy/http_aws_es/connector.js#L84-L91 

Added lines #L84 - L91 were not covered by tests

Check warning on line 93 in src/plugins/data_source/server/legacy/http_aws_es/connector.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/plugins/data_source/server/legacy/http_aws_es/connector.js#L93 

Added line #L93 was not covered by tests

Check warning on line 103 in src/plugins/data_source/server/legacy/http_aws_es/connector.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/plugins/data_source/server/legacy/http_aws_es/connector.js#L100-L103 

Added lines #L100 - L103 were not covered by tests

Check warning on line 121 in src/plugins/data_source/server/legacy/http_aws_es/connector.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/plugins/data_source/server/legacy/http_aws_es/connector.js#L119-L121 

Added lines #L119 - L121 were not covered by tests

Check warning on line 130 in src/plugins/data_source/server/legacy/http_aws_es/connector.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/plugins/data_source/server/legacy/http_aws_es/connector.js#L129-L130 

Added lines #L129 - L130 were not covered by tests

Check warning on line 17 in src/plugins/data_source/server/util/credential_provider.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/plugins/data_source/server/util/credential_provider.ts#L16-L17 

Added lines #L16 - L17 were not covered by tests
View more details on Codecov