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

AEAD support in enc app #471

Closed
zakjan opened this issue Nov 14, 2015 · 6 comments
Closed

AEAD support in enc app #471

zakjan opened this issue Nov 14, 2015 · 6 comments
Labels
issue: feature request The issue was opened to request a feature

Comments

@zakjan
Copy link

zakjan commented Nov 14, 2015

Is there any particiular reason, why AEAD ciphers are not supported in the enc app? https://github.com/openssl/openssl/blob/master/apps/enc.c#L298

I have implemented as a small part of my thesis, maybe I could polish it and submit a PR.

@mouse07410
Copy link
Contributor

It's been two months - any comments? I too find it strange that enc does not support AEAD.

@mattcaswell mattcaswell added the issue: feature request The issue was opened to request a feature label May 16, 2016
@mattcaswell mattcaswell modified the milestones: 1.1.0, Post 1.1.0 May 16, 2016
@nrgiii
Copy link
Contributor

nrgiii commented Jan 19, 2017

+1. This would be helpful for me too, particularly for OCB cipher (aes-256-ocb, etc)

@chr0n1x
Copy link

chr0n1x commented Dec 21, 2017

+1
bump

@paulidale
Copy link
Contributor

See also: #4844 which is incomplete (only GCM) and there are some questions raised about the behaviour on failure and if GCM should be used for storage at all.

@richsalz
Copy link
Contributor

I was just about to say: how to handle failure? Not an easy issue, but it needs to be addressed before this can go.

@kaduk
Copy link
Contributor

kaduk commented Jan 9, 2018

There is some more recent discussion on #5026 , and several other issues/pull-requests on this topic.
It is clearly a recurring requested item, but I think we should take a principled stand of not offering a footgun in the form of a utility for streaming AEAD output prior to authentication.

Accordingly, I propose to:

  • Add a FAQ entry on the website about AEADs and the enc(1) utility
  • Add a note to the enc(1) manual stating that AEAD modes are not and will not be supported due to the issue of having already streamed data in case of verification failure, with a reference to use cms(1) instead

Neither of those need to be addressed before the 1.1.1 release (but both could be done before that release if they are ready), and more importantly, if we did want to add this functionality, it need not be in the 1.1.1 release, so I will apply the post-1.1.1 milestone.

@kaduk kaduk modified the milestones: Post 1.1.0, Post 1.1.1 Jan 9, 2018
kaduk added a commit to kaduk/openssl that referenced this issue Jan 9, 2018
zakjan added a commit to zakjan/openssl that referenced this issue Jan 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue: feature request The issue was opened to request a feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants