Skip to content

Commit

Permalink
Merge pull request #234 from ttomsu/google-startup-accounts
Browse files Browse the repository at this point in the history
Removes excess call to Clouddriver to get accounts for anonymous auth…
  • Loading branch information
Travis Tomsu committed Jun 1, 2016
2 parents 8991054 + 1fd9486 commit 9fbe3ff
Showing 1 changed file with 3 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,8 @@

package com.netflix.spinnaker.gate.security.anonymous

import com.netflix.spinnaker.gate.services.AccountsService
import com.netflix.spinnaker.gate.security.SpinnakerAuthConfig
import com.netflix.spinnaker.security.User
import org.springframework.beans.factory.annotation.Autowired
import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean
import org.springframework.context.annotation.Configuration
import org.springframework.security.config.annotation.web.builders.HttpSecurity
Expand All @@ -30,17 +28,14 @@ import org.springframework.security.config.annotation.web.servlet.configuration.
@Configuration
@EnableWebMvcSecurity
class AnonymousConfig extends WebSecurityConfigurerAdapter {
String key = "spinnaker-anonymous"
String defaultEmail = "anonymous"

@Autowired
AccountsService accountsService
static String key = "spinnaker-anonymous"
static String defaultEmail = "anonymous"

void configure(HttpSecurity http) {
def principal = new User(
email: defaultEmail,
roles: ["anonymous"],
allowedAccounts: accountsService.getAllowedAccounts([] /* userRoles */)
allowedAccounts: []
)

http.anonymous()
Expand Down

0 comments on commit 9fbe3ff

Please sign in to comment.