#2790 Extra newline in xml certificate output
Closed: fixed Opened by rcritten.

Filed due to https://pagure.io/certmonger/issue/76

The crux of the issue is that before the -----END CERTIFICATE----- there is a blank newline which makes OpenSSL throw up.

What I see is in the field an otherwise valid certificate there is just an extra newline.

I've attached the full xml I got back after the cert is renewed.

I got this xml by hacking the life out of certmonger. I don't know how you can easily reproduce that. I'm hoping it's fairly easy on your end.

You can see the final output via:

  • Install IPA
  • modify /usr/libexec/certmonger/dogtag-ipa-ca-renew-agent-submit
  • print request_cert() after result = run(...) save the value of stdout in some file
  • getcert resubmit -I

You'll end up with a cert with the extra newline.


Metadata Update from @mharmsen:
- Custom field component adjusted to None
- Custom field feature adjusted to None
- Custom field origin adjusted to None
- Custom field proposedmilestone adjusted to None
- Custom field proposedpriority adjusted to None
- Custom field reviewer adjusted to None
- Custom field type adjusted to None
- Custom field version adjusted to None
- Issue set to the milestone: 0.0 NEEDS_TRIAGE

Metadata Update from @vakwetu:
- Issue priority set to: critical
- Issue set to the milestone: 10.5 (was: 0.0 NEEDS_TRIAGE)

Metadata Update from @edewata:
- Issue assigned to edewata

Possible fix:

  • https://review.gerrithub.io/#/c/383665/

I've created a COPR build that contains the above patch:

  • https://copr.fedorainfracloud.org/coprs/edewata/pki-10.5/package/pki-core/

Could you please retest with the above build? Thanks.

Looks like it is working to me.

Thanks!

Fixed in master:

  • https://github.com/dogtagpki/pki/commit/d38415bbf7d3be9a91d0bf029c1716e4c3e2d068

Metadata Update from @edewata:
- Issue close_status updated to: fixed
- Issue set to the milestone: 10.5.0 (was: 10.5)
- Issue status updated to: Closed (was: Open)

Metadata Update from @mharmsen:
- Issue set to the milestone: 10.5.1 (was: 10.5.0)

Metadata Update from @mharmsen:
- Custom field fixedinversion adjusted to pki-core-10.5.1-1.fc27

Dogtag PKI is moving from Pagure issues to GitHub issues. This means that existing or new
issues will be reported and tracked through Dogtag PKI's GitHub Issue tracker.

This issue has been cloned to GitHub and is available here:
https://github.com/dogtagpki/pki/issues/2910

If you want to receive further updates on the issue, please navigate to the
GitHub issue and click on Subscribe button.

Thank you for understanding, and we apologize for any inconvenience.

Metadata