Closed Bug 313583 Opened 20 years ago Closed 15 years ago

Remove long_list.cgi, showattachment.cgi and xml.cgi

Categories

(Bugzilla :: Bugzilla-General, enhancement)

Product:
Bugzilla
Component:
Bugzilla-General
Version:
2.21
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Bugzilla 4.2

People

(Reporter: LpSolit, Assigned: LpSolit)

References

(Blocks 1 open bug)

Details

(Whiteboard: [relnote in 4.0 that these scripts will go away in 4.2])

Attachments

(2 files)

patch, v1
4.52 KB, patch
mkanat
: review+
Details | Diff | Splinter Review
relnote for 4.0, v1
716 bytes, patch
mkanat
: review+
Details | Diff | Splinter Review
These are old, unused files. They should go away, at least for Bugzilla 3.0.
Breaking URLs is bad.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → WONTFIX
Target Milestone: Bugzilla 3.0 → ---
4 years later, are you still against this? We are talking about scripts which are no longer used since... Bugzilla 2.18 or even earlier.
Well, if there is some maintenance effort required to keep them around, then we should consider removing them for 4.0. If there's no maintenance effort right now, I don't see any reason to remove them.
(In reply to comment #3) > Well, if there is some maintenance effort required to keep them around, then > we should consider removing them for 4.0. If there's no maintenance effort > right now, I don't see any reason to remove them. Well, the security bug 591165 seems a good enough reason to remove them, see bug 591165 comment 15.
Status: RESOLVED → REOPENED
Flags: blocking4.0?
Resolution: WONTFIX → ---
Target Milestone: --- → Bugzilla 4.0
You know, one way or another, this is Bugzilla 4.0--it's as good a time to remove them as any. Not a blocker, but I suppose can can do it now. Might as well clean up.
Flags: blocking4.0? → blocking4.0-
Blocks: bz-clients
Attached patch patch, v1 — — Splinter Review
Assignee: general → LpSolit
Status: REOPENED → ASSIGNED
Attachment #470571 - Flags: review?(mkanat)
Igor, Rob, if we remove xml.cgi, will the desktop clients break?
(In reply to comment #7) > Igor, Rob, if we remove xml.cgi, will the desktop clients break? Max, thanks for asking! Deskzilla will break, but we'll be able to fix that. Right now it first tries xml.cgi (to accommodate old versions), but when redirect to show_bug.cgi?ctype=xml is detected, it starts going there directly without xml.cgi. Igor
For the record, note that Bugzilla 2.17.1 was the last version to display XML data directly from xml.cgi. Starting with 2.17.2, it was simply redirecting to show_bug.cgi.
Do you have any <usage by version> numbers? We're still supporting Bugzilla versions as old as 2.14, which is probably a waste of effort (not that big though).
While I'm on it: showattachment.cgi is simply a redirect since Bugzilla 2.16rc1 (meaning that 2.14.5 was the last version to effectively use this script), and long_list.cgi is a redirect since Bugzilla 2.20rc1 (so 2.19.2 was the last version to use this script). About stats, I have none, but I doubt a lot of people still use Bugzilla 2.14.
Keywords: relnote
Whiteboard: [relnote in 4.0 that these scripts will go away in 4.2]
Attachment #497920 - Flags: review?(mkanat)
Attachment #497920 - Flags: review?(mkanat) → review+
Attachment #470571 - Flags: review?(mkanat) → review+
Flags: approval+
Target Milestone: Bugzilla 4.0 → Bugzilla 4.2
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/ deleted long_list.cgi deleted showattachment.cgi deleted xml.cgi modified template/en/default/pages/release-notes.html.tmpl Committed revision 7624. Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.0/ modified template/en/default/pages/release-notes.html.tmpl Committed revision 7497.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago → 15 years ago
Resolution: --- → FIXED
Added to relnotes in bug 713346.
Keywords: relnote
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: