Error getting non Default credentials after Manifest v3 change about samltoawsstskeys HOT 7 CLOSED

@DavidRHoffman That looks like a different issue. I've created a new issue for this one. #61

from samltoawsstskeys.

Hi @hrast01, thanks for reporting. Could you provide some more of the logs prior to this error?

from samltoawsstskeys.

The issue is caused when more than one role is available. The latest changes in the scripts.js file assignes the the incorrect object to the attributes_role_list_item variable. It should return the #text element instead like the following.

attributes_role_list_item = attributes_role_list[i]['#text'];

from samltoawsstskeys.

Ok I see, so you are saying this issue is simply a result of the first AssumeRoleWithSAML call not working because of issue #58 ? That makes sense. Lets first wait on #58 getting fixed then, before continuing on this one.

PS
Using attributes_role_list[i]['#text'] is not the complete solution. This would break the extension for others. Its a little bit more complicated than that. The issue lies with the fact there are various ways to create the SAML Assertion xml. This results in different behaviour of the xml parser. But I'll get it sorted soon.

from samltoawsstskeys.

@nickcampau , b198466 implements a changed configuration of the xmlparser. This allows for a consistent way of processing the claimed IAM roles. This means I was now able to change the code to referencing the #text key as suggested.

As mentioned in #58 , I've published v3.1 to the web store. As soon as Google finishes the review, the new version will be released. I'll keep using #58 as main discussion thread, therefore I'll close this issue.

@hrast01 , obviously you can reopen this issue if the above issue somehow remains while using v3.1.

from samltoawsstskeys.

@prolane - I've got the v.3.1 update. With v3.0 I was getting the same error as was reported for #58, but now I am getting this error. Unless I am missing something, I don't think this has been resolved.

from samltoawsstskeys.

from samltoawsstskeys.