Skip to content

Commit

Permalink
fix(cloudfoundry): revert #5150 (#5171) (#5177)
Browse files Browse the repository at this point in the history
(cherry picked from commit 5acafb5)

Co-authored-by: Nicolas Cohen <ncknt@users.noreply.github.com>
  • Loading branch information
mergify[bot] and ncknt committed Jan 5, 2021
1 parent 4c2560c commit 84b2faf
Showing 1 changed file with 44 additions and 33 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,12 @@
import com.netflix.spinnaker.clouddriver.security.CredentialsInitializerSynchronizable;
import com.netflix.spinnaker.credentials.*;
import com.netflix.spinnaker.credentials.definition.AbstractCredentialsLoader;
import com.netflix.spinnaker.credentials.definition.BasicCredentialsLoader;
import com.netflix.spinnaker.credentials.definition.CredentialsDefinitionSource;
import com.netflix.spinnaker.credentials.poller.Poller;
import java.util.concurrent.ForkJoinPool;
import javax.annotation.Nullable;
import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
import org.springframework.context.ApplicationContext;
import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration;

Expand All @@ -46,38 +47,48 @@ public ForkJoinPool cloudFoundryThreadPool(
}

@Bean
public CredentialsTypeBaseConfiguration<
CloudFoundryCredentials, CloudFoundryConfigurationProperties.ManagedAccount>
cloudfoundryCredentials(
ApplicationContext applicationContext,
CloudFoundryConfigurationProperties configurationProperties,
CacheRepository cacheRepository,
ForkJoinPool cloudFoundryThreadPool) {
return new CredentialsTypeBaseConfiguration<>(
applicationContext,
CredentialsTypeProperties
.<CloudFoundryCredentials, CloudFoundryConfigurationProperties.ManagedAccount>builder()
.type(CloudFoundryProvider.PROVIDER_ID)
.credentialsClass(CloudFoundryCredentials.class)
.credentialsDefinitionClass(CloudFoundryConfigurationProperties.ManagedAccount.class)
.defaultCredentialsSource(configurationProperties::getAccounts)
.credentialsParser(
a ->
new CloudFoundryCredentials(
a.getName(),
a.getAppsManagerUri(),
a.getMetricsUri(),
a.getApi(),
a.getUser(),
a.getPassword(),
a.getEnvironment(),
a.isSkipSslValidation(),
a.getResultsPerPage(),
cacheRepository,
a.getPermissions().build(),
cloudFoundryThreadPool,
a.getSpaceFilter()))
.build());
@ConditionalOnMissingBean(
value = CloudFoundryCredentials.class,
parameterizedContainer = AbstractCredentialsLoader.class)
public AbstractCredentialsLoader<CloudFoundryCredentials> cloudFoundryCredentialsLoader(
@Nullable
CredentialsDefinitionSource<CloudFoundryConfigurationProperties.ManagedAccount>
cloudFoundryCredentialSource,
CloudFoundryConfigurationProperties configurationProperties,
CacheRepository cacheRepository,
CredentialsRepository<CloudFoundryCredentials> cloudFoundryCredentialsRepository,
ForkJoinPool cloudFoundryThreadPool) {

if (cloudFoundryCredentialSource == null) {
cloudFoundryCredentialSource = configurationProperties::getAccounts;
}
return new BasicCredentialsLoader<>(
cloudFoundryCredentialSource,
a ->
new CloudFoundryCredentials(
a.getName(),
a.getAppsManagerUri(),
a.getMetricsUri(),
a.getApi(),
a.getUser(),
a.getPassword(),
a.getEnvironment(),
a.isSkipSslValidation(),
a.getResultsPerPage(),
cacheRepository,
a.getPermissions().build(),
cloudFoundryThreadPool,
a.getSpaceFilter()),
cloudFoundryCredentialsRepository);
}

@Bean
@ConditionalOnMissingBean(
value = CloudFoundryCredentials.class,
parameterizedContainer = CredentialsRepository.class)
public CredentialsRepository<CloudFoundryCredentials> cloudFoundryCredentialsRepository(
CredentialsLifecycleHandler<CloudFoundryCredentials> eventHandler) {
return new MapBackedCredentialsRepository<>(CloudFoundryProvider.PROVIDER_ID, eventHandler);
}

@Bean
Expand Down

0 comments on commit 84b2faf

Please sign in to comment.