add back GCM cipher #67

Merged
merged 1 commit into from Jan 12, 2015

Conversation

Projects
None yet
3 participants
@arlimus
Member

arlimus commented Jan 12, 2015

we originally removed GCM from the cipher chain due to vulnerabilities in OpenSSH. We are adding it back, since any sensible implementation will have had enough time to fix or backport. On top of that, GCM is generally a preferred algorithm. Let's get it back now.

Signed-off-by: Dominik Richter dominik.richter@gmail.com

add back GCM cipher
we originally removed GCM from the cipher chain due to vulnerabilities in OpenSSH. We are adding it back, since any sensible implementation will have had enough time to fix or backport. On top of that, GCM is generally a preferred algorithm. Let's get it back now.

Signed-off-by: Dominik Richter <dominik.richter@gmail.com>
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jan 12, 2015

Coverage Status

Coverage remained the same when pulling 32f14ac on gcm into dcfc0a8 on master.

Coverage Status

Coverage remained the same when pulling 32f14ac on gcm into dcfc0a8 on master.

@chris-rock

This comment has been minimized.

Show comment
Hide comment
@chris-rock

chris-rock Jan 12, 2015

Member

great work @arlimus

Member

chris-rock commented Jan 12, 2015

great work @arlimus

chris-rock added a commit that referenced this pull request Jan 12, 2015

Merge pull request #67 from TelekomLabs/gcm
add back GCM cipher

@chris-rock chris-rock merged commit c23a1c0 into master Jan 12, 2015

1 of 2 checks passed

default Build #84 failed in 16 min
Details
continuous-integration/travis-ci The Travis CI build passed
Details

@arlimus arlimus deleted the gcm branch Jan 12, 2015

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