Closed Bug 234171 Opened 16 years ago Closed 16 years ago

Remove %COOKIE from index.cgi

Categories

(Bugzilla :: Bugzilla-General, defect)

2.17.6
defect
Not set

Tracking

()

RESOLVED FIXED
Bugzilla 2.18

People

(Reporter: wicked, Assigned: wicked)

References

Details

Attachments

(1 file, 1 obsolete file)

Change index.cgi to not use %COOKIE as per bug 225818.
Blocks: 225818
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attached patch Changes, v1 (obsolete) — Splinter Review
Attachment #141343 - Flags: review?(justdave)
Comment on attachment 141343 [details] [diff] [review]
Changes, v1

- Use user.login rather than user.email
- Check your "none" INTERFACE comment for consistency with other similar
INTERFACE comments.

Then, r=gerv. Nice :-)

Gerv
Attachment #141343 - Flags: review?(justdave) → review+
Agreed, good job!
Flags: approval+
Target Milestone: --- → Bugzilla 2.18
Attached patch Changes, v2Splinter Review
- Changed user.login to user.email
- Changed interface comment to one that is used in most interfaceless templates
(there are two templates using none and one that has no text) and which I
personally like the best

As I have only read-only privs to CVS, somebody with the Power should commit.
Attachment #141343 - Attachment is obsolete: true
Checking in index.cgi;
/cvsroot/mozilla/webtools/bugzilla/index.cgi,v  <--  index.cgi
new revision: 1.12; previous revision: 1.11
done
Checking in template/en/default/index.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/index.html.tmpl,v  <-- 
index.html.tmpl
new revision: 1.14; previous revision: 1.13
done
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Blocks: 236678
No longer blocks: 225818
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.