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

Set DH timeout to accommodate low performance CPU #317

Merged
merged 1 commit into from
Dec 12, 2018

Conversation

dspinellis
Copy link
Contributor

Set DH timeout to accommodate computers with a low performance CPU

On a Raspberry Pi B+ the Diffie-Hellman parameter generation for
a 2048 key was measured to take 229m45.379s wall clock time.
Set the respective command's timeout 50% above this, to prevent
the corresponding rule from timing out.

This Pull Request (PR) fixes the following issues

Fixes #316

Copy link
Member

@Dan33l Dan33l left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hi @dspinellis thank you for the PR.

Can you have a look at the inline comment ?
Excepted this, the PR LGTM.

manifests/ca.pp Show resolved Hide resolved
@Dan33l Dan33l added the enhancement New feature or request label Dec 11, 2018
@dspinellis
Copy link
Contributor Author

Good point; I added it.

@Dan33l
Copy link
Member

Dan33l commented Dec 12, 2018

@dspinellis please squash your commits and the PR will be merged when CI will be successfully finished.

On a Raspberry Pi B+ the Diffie-Hellman  parameter generation for
a 2048 key was measured to take 229m45.379s wall clock time.
Set the respective command's timeout 50% above this, to prevent
the corresponding rule from timing out.

Add missing timeout for EasyRSA 3.0
Copy link
Member

@Dan33l Dan33l left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Dan33l Dan33l merged commit cc6ffc2 into voxpupuli:master Dec 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Timeout when generating Diffie-Hellman parameters on a low-performance CPU
2 participants