-
Notifications
You must be signed in to change notification settings - Fork 94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
vCenter 7.0.3 Support #13
base: main
Are you sure you want to change the base?
Conversation
…hentication. Added RelayState parameter to add support for vCenter 7.0.3. This will possibly break compatibility with older versions and should be tested.
Hey! I really appreciate the information and pull request. We had reports that 7.0.3 had broken the flow, but hadn't hadn't had time to investigate just yet so this is really helpful! I have access to a few older instances, so will test the change and work on a solution to get this in. Thanks again! |
I saw those issues! That's the reason I figured I would just ship over the changes I made to get it to work. It's hacky and will probably break something else, but it worked for my instance. A bit more info about the RelayState parameter in case it's not in older versions. It acts as a sort of CSRF token where it provides you the parameter value in the initial SAML request and you provide it back when you provide the SAML response. I no longer have access to that version of vCenter so that's about the extent of what I can provide, but feel free to ask any questions to get this working for newer versions. Cheers! |
Hello, sorry to dust off an old issue but I came across this exact issue when attacking a 7.0.3 server (with permission of course :-) today. I was getting the "expected 302 redirect" and tried out @Xerzzul 's modification of the script and it worked (I think!). The script completes and I get the
When I visit https://ip.of.the.vc, I get a I'm trying to figure out if I'm chasing my tail but would LOVE if someone could help me troubleshoot? |
Did you ever figured this out ? |
I didn't - had to find another attack path :-( |
@thodorian @7MinSec From my testing I got HTTP 200 and the The only difference I found is that the signed XML from metasploit includes |
I was recently on an engagement and identified a vCenter 7.0.3 server. After troubleshooting this script I determined that this version requires a POST parameter RelayState for successful authentication. This change adds the RelayState parameter to add support for vCenter 7.0.3.
This could break compatibility with older versions and should be tested.