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

[Backport][ipa-4-6] ipa-pki-retrieve-key: ensure we do not crash #1074

Closed
wants to merge 1 commit into from

Conversation

stlaz
Copy link
Contributor

@stlaz stlaz commented Sep 13, 2017

This PR was opened automatically because PR #996 was pushed to master and backport to ipa-4-6 is required.

If ipa-pki-retrieve-key fails for some reason (which may be a
"legitimate" reason, e.g. the server it is attempting to contact
being offline), the program terminates with an uncaught exception,
resulting in crash report.

Catch all exceptions; if an exception gets raised, report the
traceback and exit with nonzero status.

Fixes: https://pagure.io/freeipa/issue/7115
@stlaz
Copy link
Contributor Author

stlaz commented Sep 13, 2017

PR was ACKed automatically because this is backport of PR #996. Wait for CI to finish before pushing. In case of questions or problems contact @frasertweedale who is author of the original PR.

@stlaz stlaz added ack Pull Request approved, can be merged re-run Trigger a new run of PR-CI labels Sep 13, 2017
@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Sep 13, 2017
@stlaz stlaz added the re-run Trigger a new run of PR-CI label Sep 13, 2017
@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Sep 13, 2017
@stlaz
Copy link
Contributor Author

stlaz commented Sep 14, 2017

ipa-4-6:

  • f9074dc ipa-pki-retrieve-key: ensure we do not crash

@stlaz stlaz added the pushed Pull Request has already been pushed label Sep 14, 2017
@stlaz stlaz closed this Sep 14, 2017
@stlaz stlaz deleted the backport_pr996_ipa-4-6 branch April 6, 2018 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants