Skip to content

Commit 616a770

Browse files
committed
tweak
1 parent 6ce0828 commit 616a770

File tree

1 file changed

+7
-3
lines changed
  • modules/repository-s3/src/main/java/org/elasticsearch/repositories/s3

1 file changed

+7
-3
lines changed

modules/repository-s3/src/main/java/org/elasticsearch/repositories/s3/S3Service.java

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -184,8 +184,7 @@ ProjectId effectiveProjectId(@Nullable ProjectId projectId) {
184184
return projectId == null ? ProjectId.DEFAULT : projectId;
185185
}
186186

187-
// visible for tests
188-
protected AmazonS3Reference buildClientReference(final S3ClientSettings clientSettings) {
187+
private AmazonS3Reference buildClientReference(final S3ClientSettings clientSettings) {
189188
final SdkHttpClient httpClient = buildHttpClient(clientSettings, getCustomDnsResolver());
190189
Releasable toRelease = httpClient::close;
191190
try {
@@ -198,8 +197,13 @@ protected AmazonS3Reference buildClientReference(final S3ClientSettings clientSe
198197
}
199198
}
200199

200+
@FixForMultiProject(description = "can be removed once blobstore is project aware")
201201
S3ClientSettings settings(RepositoryMetadata repositoryMetadata) {
202-
return s3ClientsManager.settingsForClient(ProjectId.DEFAULT, repositoryMetadata);
202+
return settings(ProjectId.DEFAULT, repositoryMetadata);
203+
}
204+
205+
S3ClientSettings settings(@Nullable ProjectId projectId, RepositoryMetadata repositoryMetadata) {
206+
return s3ClientsManager.settingsForClient(effectiveProjectId(projectId), repositoryMetadata);
203207
}
204208

205209
// proxy for testing

0 commit comments

Comments
 (0)