Git Product home page Git Product logo

Comments (11)

hrast01 avatar hrast01 commented on August 28, 2024

Tested that reverting to Chrome 67 did, in fact, resolve the issue.

from samltoawsstskeys.

prolane avatar prolane commented on August 28, 2024

I can't confirm the issue yet.
Tested with Chrome Version 68.0.3440.84 on Mac and the credentials file downloads without an issue.

Please provide debug logs.
Go to Extensions, turn on 'Developer Mode', and then click 'Inspect views background/background.html' for this extension. This will give you a new window with the console open. Try to use the extension and please share whatever output is generated.

from samltoawsstskeys.

hrast01 avatar hrast01 commented on August 28, 2024

I am also on Mac running High Sierra (10.13.6) and Chrome 68.0.3440.84. Here's what pops up in the Console in Dev mode.

Error in event handler for webRequest.onBeforeRequest/1: TypeError: Cannot read property '0' of undefined at onBeforeRequestEvent (chrome-extension://ekniobabpcnfjgfbphhcolcinmnbehde/background/script.js:88:55)

from samltoawsstskeys.

prolane avatar prolane commented on August 28, 2024

Got it, thanks. I think I see the issue.
Am I right your SAML provider only sends along one IAM role?

from samltoawsstskeys.

prolane avatar prolane commented on August 28, 2024

Never mind. I was already able to reproduce myself. The issue is indeed when just one IAM role is present in the SAML claim.

I wonder how this ever worked before Chrome 68. I guess the previous versions were a bit less restrictive to non existing indexes. The issue is in these lines of code:

if (details.requestBody.formData) {
SAMLAssertion = details.requestBody.formData.SAMLResponse[0];
hasRoleIndex = "roleIndex" in details.requestBody.formData;
roleIndex = details.requestBody.formData.roleIndex[0];

If there is no roleIndex, it can't get its value. On line 87 its actually checked if this key exists. But unfortunately on line 88 the value of the key is requested regardless of the existence of the key.

I'll create a fix for it soon.

from samltoawsstskeys.

hrast01 avatar hrast01 commented on August 28, 2024

In my case, I'm only requesting the one role's credentials, but in my colleague's case he was requesting 3 additional role's credentials.

from samltoawsstskeys.

hrast01 avatar hrast01 commented on August 28, 2024

Just for completeness (and my own sense of sanity), I added three additional roles that I can assume to the plugin, same error:

Error in event handler for webRequest.onBeforeRequest/1: TypeError: Cannot read property '0' of undefined
    at onBeforeRequestEvent (chrome-extension://ekniobabpcnfjgfbphhcolcinmnbehde/background/script.js:88:55)

from samltoawsstskeys.

prolane avatar prolane commented on August 28, 2024

Thanks for the info, but this still makes sense. The additional roles can only be assumed if the IAM role from the SAML claim can be assumed.

What I was talking about is when your Identity Provider (SAML Provider) adds multiple role claims into the SAML data.

No worries, I've already made a patch and this seems to work locally. I'll soon push it to the Chrome store.

from samltoawsstskeys.

prolane avatar prolane commented on August 28, 2024

@hrast01 New version is pushed to the Chrome store. Usually its available within 1 hour.
In order for Chrome to fetch the latest version you can best restart Chrome.

from samltoawsstskeys.

hrast01 avatar hrast01 commented on August 28, 2024

Works great, thanks for the quick update!

from samltoawsstskeys.

prolane avatar prolane commented on August 28, 2024

Thanks for letting me know!

from samltoawsstskeys.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.