-
Notifications
You must be signed in to change notification settings - Fork 423
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
Using AES instead of DES-192 for Encryption #821
Comments
It's a default paramenter in the code, here:
I think that we could use an option in the general configuration to handle this behaviour. consider also the enc template |
So does the code even support AES ? will changing key_type from des-192 to say AES-128 work? @peppelinux |
Try It, go ahead and share results, we can work on a pull request for that |
I agree that we should review this and change it (or make it configurable) |
@c00kiemon5ter / @peppelinux do you have any plans considering this issue? |
I don't have any updates, formerly we should assign this issue to a developer and this should present a PR. Is there any candidates? Hope to talk about this during the next idpy call, thank you melanger for having pointed out this |
When encrypting the SAML message, the configuration passed top
xmlsec
seem to use DES-192This is the command thats ran
Is it possible to use AES instead of DES-192, it seems like DES is hardcoded into the code and I cant find where to change it from the requirements.
Code Version
pysaml2==7.0.1
Expected Behavior
Have some way of configuring which algorithm is used for encryption
The text was updated successfully, but these errors were encountered: