Skip to content

Commit

Permalink
Merge pull request #494 from guardian/fix-panda-update-mistakes
Browse files Browse the repository at this point in the history
Fix minor panda update mistakes
  • Loading branch information
emdash-ie authored Feb 4, 2025
2 parents 25eb1cf + 4b34d36 commit 64e6f09
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion app/AppComponents.scala
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ class AppComponents(context: Context)
val panDomainRefresher = PanDomainAuthSettingsRefresher(
domain = config.domain,
system = config.pandaSystem,
S3BucketLoader.forAwsSdkV1(AWS.S3Client, "pan-domain-auth-settings")
S3BucketLoader.forAwsSdkV1(AWS.S3Client, config.pandaBucketName)
)

val permissions: PermissionsProvider =
Expand Down
1 change: 0 additions & 1 deletion app/config/Config.scala
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,6 @@ class Config(playConfig: Configuration) extends AwsInstanceTags with Logging {

lazy val pandaSystem: String = "workflow"
lazy val pandaBucketName: String = "pan-domain-auth-settings"
lazy val pandaSettingsFile: String = s"$domain.settings"

lazy val loggingStreamName: Option[String] = playConfig.getOptional[String]("aws.kinesis.logging.streamName")
lazy val loggingRole: Option[String] = playConfig.getOptional[String]("aws.kinesis.logging.stsRoleToAssume")
Expand Down

0 comments on commit 64e6f09

Please sign in to comment.