Closed (fixed)
Project:
Hosting
Version:
6.x-0.4-alpha1
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Sep 2009 at 17:10 UTC
Updated:
12 Oct 2009 at 10:30 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Anonymous (not verified) commentedI can reproduce this. Thanks for the bug report!
Comment #2
Anonymous (not verified) commentedI've committed a stopgap to cvs that takes the first port from the list of allowed ports on the web server and assigns this port when importing a site. This will need to be improved in the future and will probably be rewritten when we have #570980: allow vhosts of the same name on a different port and #588424: Default port for web servers
But for now it's fixed. If you're not running against HEAD, attached is a patch. There's a hook_update that will go and set any existing sites to the 'default' port if they are port 0 in the db.
Note that apache vhost configs were still written with port 80 for these sites, the reason they are 0 in the database is because the value was simply not being passed, so null.
Comment #3
steveparks commentedI can confirm that HEAD now updates all sites to Port 80 that were previously Port 0.
Thanks Mig5!
Steve