Closed Bug 237864 Opened 21 years ago Closed 21 years ago

Can't locate Token.pm in @INC when trying to log in

Categories

(Bugzilla :: Bugzilla-General, defect)

2.17.6
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 2.18

People

(Reporter: justdave, Assigned: justdave)

References

Details

(Keywords: regression)

Attachments

(1 file)

This is a regression from the bug 192516 checkin. I missed some occurances of Token.
Bugzilla/Auth/DB.pm:53: require Token; Bugzilla/Auth/DB.pm:54: Token::DeletePasswordTokens($userid, "user_logged_in"); Bugzilla/Auth.pm:178:retrieved through the L<Token interface|Token>. Bugzilla/Token.pm:29:# Bundle the functions in this file together into the "Token" package. Bugzilla/Token.pm:250: Token::Cancel($token, $reason); token.cgi:160: Token::IssuePasswordToken($cgi->param('loginname')); token.cgi:239: $vars->{'email'} = $new_email; # Needed for Token::Cancel's mail token.cgi:240: Token::Cancel($::token,"account_exists");
Attached patch PatchSplinter Review
Attachment #144208 - Flags: review?(gerv)
Flags: blocking2.18+
Target Milestone: --- → Bugzilla 2.18
Comment on attachment 144208 [details] [diff] [review] Patch r=gerv as long as you check none of the changes break the 80 character line length limit. Gerv
Attachment #144208 - Flags: review?(gerv) → review+
Yep, it fits (barely) Checking in token.cgi; /cvsroot/mozilla/webtools/bugzilla/token.cgi,v <-- token.cgi new revision: 1.23; previous revision: 1.22 done Checking in Bugzilla/Auth.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Auth.pm,v <-- Auth.pm new revision: 1.3; previous revision: 1.2 done Checking in Bugzilla/Token.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Token.pm,v <-- Token.pm new revision: 1.22; previous revision: 1.21 done Checking in Bugzilla/Auth/DB.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Auth/DB.pm,v <-- DB.pm new revision: 1.5; previous revision: 1.4 done
Status: NEW → RESOLVED
Closed: 21 years ago
Flags: approval+
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: