← Back to context

Comment by bawolff

5 hours ago

My favourite SAML horror story, is that it used to be, that by default the main c implementation of xmlsig would not just check the sig with the public key specified but would also:

- check it against an hmac using a password specified in the attacker controlled document.

- check the signature using web pki (so the attacker could sign the saml document with their TLS key for their own personal domain and it would always be considered valid)

I honestly dont know how sites with saml arent getting hacked all the time. The only thing worse than the absolute terrible standards are the absolute terrible implementations.

I saw multiple implementations that looked for a signature, verified it, then just trusted the document as a whole rather than only the part that was signed. So as long as you had any signed SAML doc, you could provide an attention of your choosing and just bundle the signed one somewhere arbitrary inside of it.

  • It really probably is the worst security specification ever written.

    • It’s also enormous, I assume from the attempt to have nominally composable parts that could be reused for other flows.

      It’s three entire specs bundled as one. One for the XML components, another for the documents you build from them, and another for the authentication flows built on top.

    • The root problem with SAML is there’s a million and one permutations to do the same thing.

      Signed assertions. Signed messages. Encrypted messages. Encrypted assertions. Sign after normalization. Sign before normalization. Encrypt then sign. Sign then encrypt.

      There’s too many ways to do too many things.

  • Thanks now I can't sleep and I need to send our pentesters who just finished pentesting seventh security patch to our SAML another email.

Didn’t JWT have a similar thing, where you could specify the algorithm to use and that included “null”?