Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

NumberLine / ReferenceProposed Change or QueryProposer
1N/AThe PDF appears to lack line numbers, so that may complicate the feedback here.

Scott Cantor

Fixed Scott Cantor - my bad, sorry (NH).

2L99, End of pg 4Nit, suggest you s/SAML 1.0/SAML 1.0 and 1.1 for completeness.
3L113, Page 5Nit, s/intent/intend
4Footnote 15Nit, s/subjected/subject
5Line 169

I think the "spirit" of pairwise IDs in SAML would make it improper to just forward them into the Internet as an OIDC claim. They were always intended by the original Liberty work as "secret" in some sense and not to be shared gratuitiously, so I think turning a pairwise ID into a non-pairwise ID through a proxy is not really appropriate unless the proxy is "inward" facing. I don't think the document is presuming that though. If the intent here is rather that the proxy be stateful and do a mapping from the inbound SAML value to an outbound pub claim, that isn't coming across as clearly as perhaps intended.

It may be a similar question in the opposite direction but I don't claim to speak for the "intent" behind the pairwise nature of the sub claim in OIDC, whereas I can speak for what the intent was in SAML.

6Line 211Nit, s/taking/taken
7Line 337Those are not in any sense "SAML Attribute names" as used by our community so I would suggest the mapping be limited to eduPerson/etc. and OIDC and leave the SAML part out of it. Essentially if you can deduce that a SAML Attribute corresponds to a given LDAP/X.500 Attribute Type, then your mapping can transit that hop and leave the SAML part implied. I disagree with string-based attribute names in a pretty deep way but if you're going to do that, I would just leave the non-string naming in SAML off to the side.
9footnote 15, P7Nit, s/the the/the
10L34, P1Define Research and Education - s/R&E/Research and Education (R&E)/
11L35, P2s/R&S/Research and Scholarship (R&S) (then remove eventual definition bracket mention from L435, P21)
12L103, P5s/. Reasoning is that/ because/