Skip to content

Conversation

@luehm
Copy link
Contributor

@luehm luehm commented Jan 22, 2025

Fixes

  • Correctly parses ARNs from EKS clusters within a govcloud AWS tenant (example: arn:aws-us-gov:eks:us-gov-west-1)

@luehm luehm marked this pull request as ready for review January 22, 2025 18:59
@ethancedwards8
Copy link
Member

I'd be happy to merge given that another person tests this. I don't really have a way to try it out.

@Theoreticallyhugo
Copy link
Collaborator

i cant test it, but the regex change really seems to just also match something with gov, so non gov users should be entirely unaffected. therefore id consider this pr safe.

@ethancedwards8 ethancedwards8 merged commit 3d9a8b1 into dracula:master Jan 23, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants