Skip to content

Comments

Fix ocsp app exit code#5998

Closed
mattcaswell wants to merge 1 commit intoopenssl:masterfrom
mattcaswell:fix-ocsp-exit-code
Closed

Fix ocsp app exit code#5998
mattcaswell wants to merge 1 commit intoopenssl:masterfrom
mattcaswell:fix-ocsp-exit-code

Conversation

@mattcaswell
Copy link
Member

If we run the ocsp command line app and the responder returns a
non-successful status code then the app should exit with a failure code.

Based on an original patch by Tatsuhiro Tsujikawa.

Fixes #2387

If we run the ocsp command line app and the responder returns a
non-successful status code then the app should exit with a failure code.

Based on an original patch by Tatsuhiro Tsujikawa.

Fixes openssl#2387
@mattcaswell mattcaswell added branch: master Applies to master branch branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) labels Apr 18, 2018
@mattcaswell
Copy link
Member Author

@tatsuhiro-t

@richsalz richsalz added the approval: done This pull request has the required number of approvals label Apr 18, 2018
@mattcaswell
Copy link
Member Author

Pushed. Thanks.

levitte pushed a commit that referenced this pull request Apr 19, 2018
If we run the ocsp command line app and the responder returns a
non-successful status code then the app should exit with a failure code.

Based on an original patch by Tatsuhiro Tsujikawa.

Fixes #2387

Reviewed-by: Rich Salz <[email protected]>
Reviewed-by: Richard Levitte <[email protected]>
(Merged from #5998)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: master Applies to master branch branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants