Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 531953 - XML output fails to include XML declaration (encoding is vitally important)
Summary: XML output fails to include XML declaration (encoding is vitally important)
Keywords:
Status: CLOSED EOL
Alias: None
Product: Dogtag Certificate System
Classification: Retired
Component: Certificate Manager
Version: 1.2
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: RHCS Maintainers
QA Contact: Ben Levenson
URL:
Whiteboard:
Depends On:
Blocks: freeipa20 532092 533287 ipa-fixes
TreeView+ depends on / blocked
 
Reported: 2009-10-29 23:45 UTC by John Dennis
Modified: 2020-03-27 20:10 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2020-03-27 20:10:39 UTC
Embargoed:


Attachments (Terms of Use)
patch (deleted)
2009-11-12 16:22 UTC, John Dennis
no flags Details | Diff

Description John Dennis 2009-10-29 23:45:12 UTC
The CMS server generates XML documents in a variety of circumstances, however it always omits the critically import xml declaration entity. The XML rfc's suggest the declaration should always be included. In particular it is critically important because the xml declaration declares the document encoding. The xml documents should be preceded by something like this:

<?xml version="1.0" encoding="UTF-8"?>

I have not verified that all text being output by CMS is in fact encoded in UTF-8, but it should be, and I assume it is.

The primary point is the receiver of the XML document should be able to parse an xml declaration and by virtue of that establish the xml version and document encoding.

Comment 1 John Dennis 2009-11-12 16:22:43 UTC
Created attachment 369247 [details]
patch

Comment 2 Andrew Wnuk 2009-11-18 00:10:55 UTC
attachment (id=369247) +awnuk

Comment 3 Andrew Wnuk 2009-11-25 23:43:33 UTC
Patch included in in bug #533979 (attachment #373874 [details]) is committed.


Note You need to log in before you can comment on or make changes to this bug.