Opened 5 years ago

Closed 5 years ago

#2249 closed change (fixed)

Support UDP sockets in multiplexer.fcgi

Reported by: matze Assignee: matze
Priority: P4 Milestone:
Module: Sitescripts Keywords:
Cc: sebastian, fred Blocked By:
Blocking: Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/4826294168584192

Description

The current version of the multiplexer.fcgi script does recognize the FCGI_BIND_ADDRESS environment variable, but it does not yet support specifying a path for an UDP socket - although class flup.server.fcgi.WSGIServer does support a /path notation here. While I do not consider this a defect, it's still a missing feature. Especially because this functionality is required to allow for emulation of the production environment, without prior manipulation of the nginx(8) configuration files.

Finally, please note that I have a working patch already, created during the tests for #2222. This ticket is merely a formality to bring that patch-set into the proper process, including review, whilst avoiding a Noissue - ... commit.

Change History (2)

comment:1 Changed 5 years ago by matze

  • Cc fred added
  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:2 Changed 5 years ago by sebastian

  • Priority changed from Unknown to P4
  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.