Raven Kong wrote:
>
>and restart mailman then fix_url on a testing purpose list 'testlist1', I
>have this verbose info:
>*Setting web_page_url to: http://a.b.c:8888/mailman/*
>*Setting host_name to: a.b.c*
>
>This still doesn't fix the problem, 'testlist1' is not displayed. Neither
>does the new 'testlist2' I created after the changes made.
>Do you have any idea about this?


It appears to be a bug (introduced by me in 2.1.13 by the fix for
<https://bugs.launchpad.net/mailman/+bug/342162>). The test for "this
host" doesn't properly account for the :port.

One issue for a fix is the fact that example.com:123 and
example.com:456 may be considered different hosts or they may not, so
I think I'll ignore that.

You have two ways to proceed. I think you said all your lists are in
one domain, so you can just set

VIRTUAL_HOST_OVERVIEW = Off

in mm_cfg.py, and all your advertised lists will display on the
overview, or you can fix Mailman/Cgi/listinfo.py (and admin.py).

I have reported this as
<https://bugs.launchpad.net/mailman/+bug/597741>, and a tentative (so
far untested) patch to fix it is attached.

-- 
Mark Sapiro <m...@msapiro.net>        The highway is for gamblers,
San Francisco Bay Area, California    better use your sense - B. Dylan

=== modified file 'Mailman/Cgi/admin.py'
--- Mailman/Cgi/admin.py        2010-03-29 20:48:11 +0000
+++ Mailman/Cgi/admin.py        2010-06-23 15:07:39 +0000
@@ -233,8 +233,9 @@
     for name in listnames:
         mlist = MailList.MailList(name, lock=0)
         if mlist.advertised:
-            if mm_cfg.VIRTUAL_HOST_OVERVIEW and \
-                   mlist.web_page_url.find('/%s/' % hostname) == -1:
+            if mm_cfg.VIRTUAL_HOST_OVERVIEW and (
+                   mlist.web_page_url.find('/%s/' % hostname) == -1 and
+                   mlist.web_page_url.find('/%s:' % hostname) == -1:
                 # List is for different identity of this host - skip it.
                 continue
             else:

=== modified file 'Mailman/Cgi/listinfo.py'
--- Mailman/Cgi/listinfo.py     2010-03-29 20:48:11 +0000
+++ Mailman/Cgi/listinfo.py     2010-06-23 15:04:41 +0000
@@ -89,8 +89,9 @@
     for name in listnames:
         mlist = MailList.MailList(name, lock=0)
         if mlist.advertised:
-            if mm_cfg.VIRTUAL_HOST_OVERVIEW and \
-                   mlist.web_page_url.find('/%s/' % hostname) == -1:
+            if mm_cfg.VIRTUAL_HOST_OVERVIEW and (
+                   mlist.web_page_url.find('/%s/' % hostname) == -1 and
+                   mlist.web_page_url.find('/%s:' % hostname) == -1):
                 # List is for different identity of this host - skip it.
                 continue
             else:

------------------------------------------------------
Mailman-Users mailing list Mailman-Users@python.org
http://mail.python.org/mailman/listinfo/mailman-users
Mailman FAQ: http://wiki.list.org/x/AgA3
Security Policy: http://wiki.list.org/x/QIA9
Searchable Archives: http://www.mail-archive.com/mailman-users%40python.org/
Unsubscribe: 
http://mail.python.org/mailman/options/mailman-users/archive%40jab.org

Reply via email to