Closed Bug 283933 Opened 20 years ago Closed 20 years ago

Bugzilla should send XML templates as application/xml

Categories

(Bugzilla :: Bugzilla-General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 2.20

People

(Reporter: annevk, Assigned: annevk)

Details

Attachments

(1 file)

The patch I'm going to attach already got review (and approval) in bug 87976
(attachment 175436 [details] [diff] [review]). It just needed to be moved to a separate bug. Do I need to
get these things again?
Attached patch patch #1Splinter Review
Assignee: general → bug
Status: NEW → ASSIGNED
Yeah, we need to get the things again. We need to just be certain that there's
no problem in any browser (or anything else that we want to serve XML to) with
application/xml.
Target Milestone: --- → Bugzilla 2.20
Attachment #175710 - Flags: review?(justdave)
Comment on attachment 175710 [details] [diff] [review]
patch #1

It's the correct mime type, if browsers choke on it, boo hiss on them.	The
ones that'll choke on it are likely the ones that wouldn't understand it
anyway, and the change to application/* will just make them download it instead
of display it, which is probably going to be the correct answer most of the
time anyway (most people won't be pulling XML from a browser anyway, but from a
script)
Attachment #175710 - Flags: review?(justdave) → review+
Flags: approval?
I meant to approve this when I reviewed it... apologies.
Flags: approval? → approval+
Checking in Bugzilla/Constants.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Constants.pm,v  <--  Constants.pm
new revision: 1.19; previous revision: 1.18
done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: