I was thinking that you need a TODO statement to remind yourself to implement it. However, you already have one at the top of the class for SAML1 and SAML2. You are good.