Skip to content

Load idp certs from file#178

Merged
pitbulk merged 4 commits intoSAML-Toolkits:masterfrom
rahulraina7:master
Jan 15, 2020
Merged

Load idp certs from file#178
pitbulk merged 4 commits intoSAML-Toolkits:masterfrom
rahulraina7:master

Conversation

@rahulraina7
Copy link
Copy Markdown

This is very trivial. But loading idp public certificate just like the sp certificates.

@pitbulk
Copy link
Copy Markdown
Contributor

pitbulk commented Jan 7, 2020

There are places in the code that uses idp_data['x509cert'] instead of the method get_idp_cert, for example:

Those should be updated to use get_idp_cert

@rahulraina7
Copy link
Copy Markdown
Author

Thanks. Updated. Also updated some of the existing tests by making use of a new settings file which doesn't have a custom-base-path, so the idp.crt is not found in the path.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants