A very minor update to flup. If you aren't having any problems now, then there's probably no need to upgrade.
This update concerns certain web servers & configurations that don't correctly supply PATH_INFO and/or QUERY_STRING to flup. When missing these environment entries, the flup servers will now attempt to deduce PATH_INFO and/or QUERY_STRING from REQUEST_URI (if it is present) before defaulting to the empty string (as it did before).
First I'd like to say that flup is awesome! However, I'm having a little bit of trouble...
I can't seem to get flup.server.fcgi to work with a unix domain socket. After investigating the problem a bit, i found this file:
which (for me, as a turbogears user) was actually controlling the creation of flup.server.fcgi I noticed that no matter what, bindAddress is set to a tuple (therefore eliminating the ability to run a server on a unix domain socket)
The simple fix that I employed was to add a conditional to line 6
if not 'bindAddress' in local_config:
which would make it so that the tuple is only created if bindAddress is empty.
Alternatively, you could provide a config value "socket"
and do something like
if 'socket' in local_config:
local_config['bindAddress'] = local_config['socket']
del local_config['socket'] #no sense storing it twice, plus socket could cause a name collision
local_conf['bindAddress'] = (host, int(port))
**** I prefer this second method as it provides a more consistent configuration value to the user
Also sorry, I would have posted a trac ticket, but I don't see a way to sign up to do so...
Thanks for the suggestion. It looks like the original flup factories (bundled with Paste) supported the 'socket' option. I didn't realize that feature was lost.
I went ahead and committed your change, but I have no idea when I'll be making the next release (I knew I was a bit hasty with yesterday's release). But the latest is always available at http://hg.saddi.com/flup-server/
And as for trac, the username/password is trac/trac1234. I should make it more prominent.