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

Avoid small primes for DH and allow rebuild of DH primes #163

Merged
merged 2 commits into from
Jan 18, 2017

Conversation

artem-sidorenko
Copy link
Member

@artem-sidorenko artem-sidorenko commented Jan 14, 2017

Fixes GH-65

Related PR to the ssh-baseline with functional tests: dev-sec/ssh-baseline#77

We have two things in this PR (separated in different commits):

  • one thing is to remove the small primes
  • second thing is to allow rebuild of primes. This takes usually a lot of time and is therefore disabled per default

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 99.79% when pulling 1c3b104 on artem-sidorenko:dh-moduli into 5fa902c on dev-sec:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 99.791% when pulling 809a7f8 on artem-sidorenko:dh-moduli into 5fa902c on dev-sec:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling a2fa759 on artem-sidorenko:dh-moduli into 5fa902c on dev-sec:master.

@coveralls
Copy link

coveralls commented Jan 14, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 8f10f45 on artem-sidorenko:dh-moduli into 5fa902c on dev-sec:master.

@artem-sidorenko
Copy link
Member Author

@atomic111 rebased as agreed

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 74d0c6f on artem-sidorenko:dh-moduli into 94c08e7 on dev-sec:master.

@atomic111
Copy link
Member

@artem-sidorenko great!!! thank you

@atomic111 atomic111 merged commit 29302d7 into dev-sec:master Jan 18, 2017
@artem-sidorenko artem-sidorenko deleted the dh-moduli branch January 18, 2017 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants