The default bug view has changed. See this FAQ.

Back out taint-related changes from bug 160710

RESOLVED FIXED in Bugzilla 2.18

Status

()

Bugzilla
User Accounts
RESOLVED FIXED
14 years ago
4 years ago

People

(Reporter: gerv, Assigned: gerv)

Tracking

2.17.3
Bugzilla 2.18
Bug Flags:
approval +

Details

Attachments

(1 attachment)

(Assignee)

Description

14 years ago
In bug 160710 we changed the nice:
-  [% PROCESS "account/prefs/${current_tab.name}.html.tmpl" IF
current_tab.name.defined %]

to a nasty hard-coded switch-like thing, for perl 5.005 taint reasons. Now we
are 5.6 only, we can restore the old code.

Gerv
(Assignee)

Comment 1

14 years ago
Created attachment 116026 [details] [diff] [review]
Patch v.1

Here we are.

Gerv
(Assignee)

Updated

14 years ago
Attachment #116026 - Flags: review?(bbaetz)
Does this work with 5.6.0? That still had some of the bugs relating to this.
(Assignee)

Comment 3

14 years ago
Er... I dunno :-) Who still uses 5.6.0 that we can get to test it?

Gerv
The jdr200 tinderbox machine has 5.6.0 on it still.  I can try it there.
Anyone with OSX, ie justdave :)
(Assignee)

Comment 6

14 years ago
Dave: are you going to be able to test this?

Gerv
Attachment #116026 - Flags: review?(justdave)
Comment on attachment 116026 [details] [diff] [review]
Patch v.1

works on 5.6.0 osx
Attachment #116026 - Flags: review?(justdave)
Attachment #116026 - Flags: review?(bbaetz)
Attachment #116026 - Flags: review+
Flags: approval+
(Assignee)

Comment 8

14 years ago
Fixed.

Checking in template/en/default/account/prefs/prefs.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/account/prefs/prefs.html.tmpl,v
 <--  prefs.html.tmpl
new revision: 1.10; previous revision: 1.9
done

Gerv
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → Bugzilla 2.18
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.