Merge pull request #1080 from adamcstephens/fix-profiles

support sso credentials with profile in section name

lib/ex_aws/credentials_ini/file.ex

@@ -227,13 +227,19 @@ if Code.ensure_loaded?(ConfigParser) do
     end
 
     def parse_ini_file({:ok, contents}, profile_name) do
+      composite_key = "profile " <> profile_name
+
       contents
       |> ConfigParser.parse_string()
       |> case do
         {:ok, %{^profile_name => config} = full} ->
           merge_special_keys(full, config)
           |> strip_key_prefix()
 
+        {:ok, %{^composite_key => config} = full} ->
+          merge_special_keys(full, config)
+          |> strip_key_prefix()
+
         {:ok, %{}} ->
           %{}
 

test/ex_aws/credentials_ini/file_test.exs

@@ -60,6 +60,29 @@ defmodule ExAws.CredentialsIni.File.FileTest do
     assert config.sso_role_name == "SomeRole"
   end
 
+  test "config file is parsed with non-default sso config profile that uses sso_session" do
+    example_config = """
+    [sso-session somecompany]
+    sso_start_url = https://start.us-gov-home.awsapps.com/directory/somecompany
+    sso_region = us-gov-west-1
+
+    [profile somecompany]
+    sso_session = somecompany
+    sso_account_id = 123456789101
+    sso_role_name = SomeRole
+    region = us-gov-west-1
+    output = json
+    """
+
+    config = ExAws.CredentialsIni.File.parse_ini_file({:ok, example_config}, "somecompany")
+
+    assert config.sso_session == "somecompany"
+    assert config.sso_start_url == "https://start.us-gov-home.awsapps.com/directory/somecompany"
+    assert config.sso_region == "us-gov-west-1"
+    assert config.sso_account_id == "123456789101"
+    assert config.sso_role_name == "SomeRole"
+  end
+
   test "{:system} in profile name gets dynamic profile name" do
     System.put_env("AWS_PROFILE", "custom-profile")
 
@@ -79,6 +102,36 @@ defmodule ExAws.CredentialsIni.File.FileTest do
     assert credentials.security_token == "TESTTOKEN"
   end
 
+  test "{:system} in profile name gets dynamic profile name using sso config" do
+    System.put_env("AWS_PROFILE", "custom-profile")
+
+    example_credentials = """
+    [sso-session somecompany]
+    sso_start_url = https://start.us-gov-home.awsapps.com/directory/somecompany
+    sso_region = us-gov-west-1
+
+    [profile custom-profile]
+    sso_session = somecompany
+    sso_account_id = 123456789101
+    sso_role_name = SomeRole
+    region = us-gov-west-1
+    output = json
+    """
+
+    credentials =
+      ExAws.CredentialsIni.File.parse_ini_file({:ok, example_credentials}, :system)
+      |> ExAws.CredentialsIni.File.replace_token_key()
+
+    assert credentials.sso_session == "somecompany"
+
+    assert credentials.sso_start_url ==
+             "https://start.us-gov-home.awsapps.com/directory/somecompany"
+
+    assert credentials.sso_region == "us-gov-west-1"
+    assert credentials.sso_account_id == "123456789101"
+    assert credentials.sso_role_name == "SomeRole"
+  end
+
   test "config file is parsed" do
     example_config = """
     [default]