Second [Change Request] Fix pkgdb (un)privileged operations for old python-bugzilla

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The change below broke bugz.fedoraproject.org.

Backing out the change to the unprivileged user but leaving the privileged
user change in appears to fix things (although we haven't tested everything
as stg doesn't talk to bugzilla as much as production does.)  Since
bugz.fedoraproject.org is fairly visible, I'd like to apply the following
hotfix to fix that.  I'll alert the cvsadmin to let me know if this
re-breaks processing of scmadmin requests.

diff --git a/modules/hotfix/files/pkgdb/utils.py b/modules/hotfix/files/pkgdb/utils.py
index 8c121c5..ef0001b 100644
--- a/modules/hotfix/files/pkgdb/utils.py
+++ b/modules/hotfix/files/pkgdb/utils.py
@@ -51,7 +51,7 @@ from cherrypy import request
 from sqlalchemy import select, and_
 from sqlalchemy.exceptions import DataError
 
-from bugzilla import RHBugzilla3
+from bugzilla import RHBugzilla3, Bugzilla
 import memcache
 
 # The Fedora Account System Module
@@ -218,7 +218,7 @@ def get_unprivileged_bz():
     bz_server = config.get('bugzilla.queryurl', config.get('bugzilla.url',
         'https://bugzilla.redhat.com'))
     bz_url = bz_server + '/xmlrpc.cgi'
-    _unpriv_bugzilla = RHBugzilla3(url=bz_url, cookiefile=None)
+    _unpriv_bugzilla = Bugzilla(url=bz_url, cookiefile=None)
     return _unpriv_bugzilla
 
 def get_bz():

-Toshio


> On Thu, 24 May 2012 08:01:28 -0700
> Toshio Kuratomi <a.badger@xxxxxxxxx> wrote:
> 
> > 
> > diff --git a/modules/hotfix/files/pkgdb/utils.py
> > b/modules/hotfix/files/pkgdb/utils.py index 021a80f..8c121c5 100644
> > --- a/modules/hotfix/files/pkgdb/utils.py
> > +++ b/modules/hotfix/files/pkgdb/utils.py
> > @@ -51,7 +51,7 @@ from cherrypy import request
> >  from sqlalchemy import select, and_
> >  from sqlalchemy.exceptions import DataError
> >  
> > -from bugzilla import Bugzilla
> > +from bugzilla import RHBugzilla3
> >  import memcache
> >  
> >  # The Fedora Account System Module
> > @@ -218,9 +218,8 @@ def get_unprivileged_bz():
> >      bz_server = config.get('bugzilla.queryurl',
> > config.get('bugzilla.url', 'https://bugzilla.redhat.com'))
> >      bz_url = bz_server + '/xmlrpc.cgi'
> > -    _unpriv_bugzilla = Bugzilla(url=bz_url, cookiefile=None)
> > +    _unpriv_bugzilla = RHBugzilla3(url=bz_url, cookiefile=None)
> >      return _unpriv_bugzilla
> > -    
> >  
> >  def get_bz():
> >      '''Retrieve a connection to bugzilla
> > @@ -236,7 +235,7 @@ def get_bz():
> >      bz_url = bz_server + '/xmlrpc.cgi'
> >      bz_user = config.get('bugzilla.user')
> >      bz_pass = config.get('bugzilla.password')
> > -    _bugzilla = Bugzilla(url=bz_url, user=bz_user, password=bz_pass,
> > +    _bugzilla = RHBugzilla3(url=bz_url, user=bz_user,
> > password=bz_pass, cookiefile=None)
> >      return _bugzilla
> > 
> > 
> > -Toshio
> 
> +1
> -sv


Attachment: pgpiQHvlv8vaC.pgp
Description: PGP signature

_______________________________________________
infrastructure mailing list
infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/infrastructure

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux