#7652 ipaserver/plugins/cert.py: Add reason to raise of errors.NotFound
Closed: fixed 5 years ago Opened 5 years ago by twoerner.

Description of problem:

In the case that enabledService is not found ipaConfigString kdc entry, a
NotFound error was raised without setting the reason. This resulted in a
traceback.
This patch fixes a trace back that is happening while deploying a replica DL1 with 4.7.0 DL1 is not able to setup PKINIT. This is another issue, but the trace back because of missing reason is fixed by this patch.

Upstream pull request: https://github.com/freeipa/freeipa/pull/2182

Version-Release number of selected component (if applicable):

freeipa-4.5.1+

How reproducible:

Always

Steps to Reproduce (with 4.7.0):

  1. Install replica DL1
  2. Long time out while setting up PKINIT
  3. Check /var/log/httpd/errors_log on replica and server

Actual results:

Additional trace back because of missing reason argument for NotFound in the logs.

Expected results:

No additional trace back because of missing reason argument for NotFound in the logs.

Additional info:

The issue is normally not seen (code patch not hit), but is possible right now with 4.7.0.


master:

  • 71ba408 ipaserver/plugins/cert.py: Added reason to raise of errors.NotFound

Metadata Update from @twoerner:
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)

5 years ago

Metadata Update from @rcritten:
- Custom field rhbz adjusted to https://bugzilla.redhat.com/show_bug.cgi?id=1609882

5 years ago

ipa-4-7:

  • b7db3ec ipaserver/plugins/cert.py: Added reason to raise of errors.NotFound

ipa-4-5:

  • 247d57f ipaserver/plugins/cert.py: Added reason to raise of errors.NotFound

ipa-4-6:

  • 70aae62 ipaserver/plugins/cert.py: Added reason to raise of errors.NotFound

Login to comment on this ticket.

Metadata