-
Notifications
You must be signed in to change notification settings - Fork 3
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
PA-22319 - Make go-saml handle signatures on response XML (Georgia Pacific) #3
PA-22319 - Make go-saml handle signatures on response XML (Georgia Pacific) #3
Conversation
@alexvalenzuelac I'm going to remove @sagard and I as reviewers until this is ready for review. |
@@ -9,9 +9,9 @@ import ( | |||
) | |||
|
|||
const ( | |||
xmlResponseID = "urn:oasis:names:tc:SAML:2.0:protocol:Response" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was moved by golangci-lint
@@ -134,3 +134,11 @@ func Decrypt(xml string, privateKeyPath string) ([]byte, error) { | |||
func deleteTempFile(filename string) { | |||
_ = os.Remove(filename) | |||
} | |||
|
|||
func LoadXml(certPath string) (string, error) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's keep this function in case if we have a Request/Response and we need to test/validate somethig with it
…ure on response, assertion and encryptedassertion
3342a0c
to
8de6d72
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for removing that cert, Alex!
Created unit tests and add additional validation for signature on