Skip to content

Commit

Permalink
OP-21723 : Reverting gate config
Browse files Browse the repository at this point in the history
  • Loading branch information
vivek-opsmx committed May 13, 2024
1 parent 2cf0495 commit 009ec53
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
package com.netflix.spinnaker.gate.security.saml

import com.netflix.spectator.api.Registry
import com.netflix.spinnaker.fiat.shared.FiatClientConfigurationProperties
//import com.netflix.spinnaker.fiat.shared.FiatClientConfigurationProperties
import com.netflix.spinnaker.gate.config.AuthConfig
import com.netflix.spinnaker.gate.security.AllowedAccountsSupport
import com.netflix.spinnaker.gate.security.SpinnakerAuthConfig
Expand Down Expand Up @@ -224,8 +224,8 @@ class SamlSsoConfig {
@Autowired
AllowedAccountsSupport allowedAccountsSupport

@Autowired
FiatClientConfigurationProperties fiatClientConfigurationProperties
// @Autowired
// FiatClientConfigurationProperties fiatClientConfigurationProperties

@Autowired
Registry registry
Expand Down Expand Up @@ -265,17 +265,18 @@ class SamlSsoConfig {
id = id.withTag("success", true).withTag("fallback", "none")
} catch (Exception e) {
log.debug(
"Unsuccessful SAML authentication (user: {}, roleCount: {}, roles: {}, legacyFallback: {})",
"Unsuccessful SAML authentication (user: {}, roleCount: {}, roles: {})",
username,
roles.size(),
roles,
fiatClientConfigurationProperties.legacyFallback,e
// fiatClientConfigurationProperties.legacyFallback,
e
)
id = id.withTag("success", false).withTag("fallback", fiatClientConfigurationProperties.legacyFallback)
id = id.withTag("success", false).withTag("fallback", false)

if (!fiatClientConfigurationProperties.legacyFallback) {
throw e
}
// if (!fiatClientConfigurationProperties.legacyFallback) {
// throw e
// }
} finally {
registry.counter(id).increment()
}
Expand Down
2 changes: 2 additions & 0 deletions gate-web/config/gate.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@ server:
compression:
enabled: true

fiat:
enabled: true

services:
deck:
Expand Down

0 comments on commit 009ec53

Please sign in to comment.