[HOTFIX request] mirrormanager] v1.4: model.py: fix user_sites() variable name (80dd827)

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

 



Fix one bug yesterday with a hotfix, only to hit a second bug later.  Here's the fix to the second bug...  This keeps a non-sysadmin user from logging into the admin interface.

+1s please.

--
Matt Domsch
Distinguished Engineer, Technology Strategist
Dell | Office of the CTO

>---------------------------------------------------------------

commit 80dd82765f337e7f9a409b4e351c848bdb5aca59
Author: Matt Domsch <Matt_Domsch@xxxxxxxx>
Date:   Fri Jun 21 07:13:39 2013 -0500

    model.py: fix user_sites() variable name


>---------------------------------------------------------------

 server/mirrormanager/model.py |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/server/mirrormanager/model.py b/server/mirrormanager/model.py index bba2ee7..6ebadc7 100644
--- a/server/mirrormanager/model.py
+++ b/server/mirrormanager/model.py
@@ -152,7 +152,7 @@ def user_sites(identity):
                                                                      SiteAdmin.q.username == identity.current.user_name)))
     for site in query_result:
         for h in site.hosts:
-            result.append((site.id, site.name, host.id, host.name))
+            result.append((site.id, site.name, h.id, h.name))
 
     sites = _min_sites_and_hosts(result)
     return sites

_______________________________________________
mirrormanager-commits mailing list
mirrormanager-commits@xxxxxxxxxxxxxxxxxxxxxx
https://lists.fedorahosted.org/mailman/listinfo/mirrormanager-commits
_______________________________________________
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