User cannot log in on PostgreSQL

RESOLVED FIXED in Bugzilla 2.20

Status

()

defect
P1
normal
RESOLVED FIXED
14 years ago
14 years ago

People

(Reporter: mkanat, Assigned: mkanat)

Tracking

2.19.2
Bugzilla 2.20
Bug Flags:
approval +

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

14 years ago
This one's pretty easy:

DBD::Pg::st execute failed: ERROR:  pg_atoi: error in " .  [for Statement
"SELECT grantor_id
                                             FROM group_group_map
                                            WHERE member_id=?
                                              AND grant_type=' . 
                                                  GROUP_MEMBERSHIP . '"] at
Bugzilla/User.pm line 480
	Bugzilla::User::derive_groups('Bugzilla::User=HASH(0x8921fec)','undef') called
at Bugzilla/User.pm line 138
	Bugzilla::User::_create('Bugzilla::User','userid=?',1) called at
Bugzilla/User.pm line 57
	Bugzilla::User::new('Bugzilla::User',1) called at Bugzilla/Auth/Login/WWW.pm
line 71
	Bugzilla::Auth::Login::WWW::login('Bugzilla::Auth::Login::WWW',2) called at
Bugzilla.pm line 64
	Bugzilla::login('Bugzilla',2) called at /var/www/html/dbinstall/query.cgi line 63

Look at the quoting.
(Assignee)

Comment 1

14 years ago
Posted patch Fix quoting (obsolete) — Splinter Review
I don't even really know how this ran on MySQL.

MySQL sure was um... nice to us. :-)
Attachment #176846 - Flags: review?(bugreport)
(Assignee)

Updated

14 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → Bugzilla 2.20
(Assignee)

Comment 2

14 years ago
Actually, I'm going to expand the scope of the bug a little.
Summary: Bugzilla::User::derive_groups cannot run on PostgreSQL → User cannot log in on PostgreSQL
(Assignee)

Comment 3

14 years ago
OK, two fixes to User.pm here. This also works on MySQL, tested.
Attachment #176846 - Attachment is obsolete: true
Attachment #176848 - Flags: review?(bugreport)
(Assignee)

Updated

14 years ago
Attachment #176846 - Flags: review?(bugreport)
(Assignee)

Updated

14 years ago
Priority: -- → P1

Comment 4

14 years ago
Comment on attachment 176848 [details] [diff] [review]
Make it possible to login to a Pg installation

r=joel by inspection
Attachment #176848 - Flags: review?(bugreport) → review+
(Assignee)

Updated

14 years ago
Flags: approval?
Flags: approval? → approval+
(Assignee)

Comment 5

14 years ago
Checking in Bugzilla/User.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/User.pm,v  <--  User.pm
new revision: 1.46; previous revision: 1.45
done
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.