refactor: use list instead of map (!25) · Merge requests · GitLab.com / GitLab Security Department / Security Operations Sub-department / Infrastructure Security

Description

Migrating this to use a list(string) instead of a map(string) to cleanup the code a little bit and make more readable.

Change var.scp_principals_map to var.scp_principals_list
Convert from map(string) to list(string)
Update IAM policy to use formatlist() instead of [ for item in ... : format() ]