Skip to content
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

Allow changing the public exponent for RSA keys #13

Closed
ionut-arm opened this issue Dec 13, 2019 · 4 comments
Closed

Allow changing the public exponent for RSA keys #13

ionut-arm opened this issue Dec 13, 2019 · 4 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@ionut-arm
Copy link
Member

In the Transient Object Context we do not allow the clients to set a custom public exponent, always defaulting to 0 (i.e. 65537).
While the specs mention that this parameter is optional for TPM implementations, this crate should ideally support customisation for any clients that want to use it.

@ionut-arm ionut-arm added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Dec 13, 2019
@hug-dev
Copy link
Member

hug-dev commented Sep 9, 2020

The RsaExponent could probably be helpful there.

@puiterwijk
Copy link
Collaborator

I'm not sure that we should support this. From a crypto point of view, I think it's easy for people to Get Crypto Wrong(R) by picking the wrong exponent.
Though from a low-level I guess we should support it.

@Superhepper
Copy link
Collaborator

I think we should support it but make it cumbersome. To avoid some one doing it by mistake.

@hug-dev
Copy link
Member

hug-dev commented Feb 3, 2021

Ok, closing for now as not something we will likely do anytime soon

@hug-dev hug-dev closed this as completed Feb 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants