Closed twisted-trac closed 20 years ago
Automation removed owner |
---|
@itamarst commented |
---|
#!html
<pre>
The idea is to allow tap plugins to just use this instead of
duplicating the "choose which port and if it's unix socket
or tcp or ssl and what interface optionally" code for each
server. Probably other uses as well.
E.g. "mktap web --listen tcp:80:127.0.0.1" or something.
Format needs to be decided.
The scheme should be pluggable so you can add new
transports. I'd argue this shouldn't be done on reactor
level (why should cReactor need to parse this?) but as a
utility function.
</pre>
@radix commented |
---|
#!html
<pre>
well, it looks fine to me. The little example section at the
bottom has the ssl bit commented out with something about
"needs a real server.pem", but it seems, from reading the
code, that it should work fine.. wouldn't you just say
ssl:80:privateKey=server.pem
?
Maybe it would also be cool to have that code that builds a
function call out of strings be moved into something like
twisted.python.util.stringToFunctionCall. Although, I guess
we can wait until we have a real use case for that :-)
</pre>
@moshez commented |
---|
#!html
<pre>
Have a look at sandbox/moshez/ports.py for a different take on the same idea --
I seperated the listening and parsing aspects, so instead of creating a
server which chooses stuff at the last moment, the service created is already
the correct type. I'm not sure where this should go, maybe twisted.application
[even though it's not necessarily applicationy stuff, but internet is also
invalid -- though the service() function could go in t.application.internet
witht he rest of ports hanging in twisted.internet
</pre>
Searchable metadata
``` trac-id__68 68 type__enhancement enhancement reporter__itamarst itamarst priority__low low milestone__ branch__ branch_author__ status__closed closed resolution__fixed fixed component__ keywords__ time__1058183898000000 1058183898000000 changetime__1065031298000000 1065031298000000 version__ owner__ cc__glyph cc__radix cc__itamarst cc__moshez ```