[GitHub] [cxf-fediz] amergey opened a new pull request #60: support validating signature whithout KeyInfo

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] [cxf-fediz] amergey opened a new pull request #60: support validating signature whithout KeyInfo

GitBox

amergey opened a new pull request #60:
URL: https://github.com/apache/cxf-fediz/pull/60


   I started a discussion, some time ago, in the mailing list, related to signature without KeyInfo (which is valid according to SAML) but not supported by Fediz. At that time I manage to configure the IDP to send KeyInfo, so I forgot about it, but recently this need come back again, but that time no ability to configure the IDP to send KeyInfo.
   
   This pull request is a proposal to support this, when KeyInfo is not provided in signature, the validation process iterates through all certificates in trust manager.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]


Reply | Threaded
Open this post in threaded view
|

[GitHub] [cxf-fediz] amergey commented on pull request #60: support validating signature whithout KeyInfo

GitBox

amergey commented on pull request #60:
URL: https://github.com/apache/cxf-fediz/pull/60#issuecomment-673489737


   done : https://issues.apache.org/jira/browse/FEDIZ-251


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]


Reply | Threaded
Open this post in threaded view
|

[GitHub] [cxf-fediz] coheigea merged pull request #60: support validating signature whithout KeyInfo

GitBox
In reply to this post by GitBox

coheigea merged pull request #60:
URL: https://github.com/apache/cxf-fediz/pull/60


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]