Colubrid WSGI Request Handler 0.8

[jens]

Jup, das war es

[jens]

Ich hab noch eine generelle Anmekrung zur Namenswahl. Warum heißt es überhaupt CGIServer in colubird?
Für mich ist ein Apache mit CGI ehr ein CGI-Server

Ich würde vorschlagen das ehr CGIbridge zu nennen. Denn es ist doch ehr eine Brücke zwischen dem ApacheCGIServer und der WSGI-Applikation. Oder vielleicht auch CGIAdapter...

EDIT:
Ich hab nochmal mit --hostname und --port gespielt... Leider funktioniert ein --hostname "" nicht.
Das liegt an options.hostname or 'localhost' Denn das Ergebniss ist 'localhost' wenn options.hostname = "" ist

Hier mal ein Vorschlag:

Code: Alles auswählen

Index: W:/testing_Server/PyLucid_WSGI/colubrid/execute.py
===================================================================
...
 class StandaloneServer(ThreadingMixIn, HTTPServer):
 
     def __init__(self, application, exports, hostname, port, noreload, quiet):
+        print "Starting standalone Server at '%s' port:%s" % (hostname, port)
         HTTPServer.__init__(self, (hostname, port), WSGIHandler)

...

class CommandlineApplication(object):
...
    def handle_parameters(self):
...

         if args[0] == 'runserver':
-            self.run_standalone(options.hostname or 'localhost',
+            if options.hostname == None: options.hostname = "localhost"
+            self.run_standalone(options.hostname,
                                 options.port or 8080, bool(options.no_reload),
                                 bool(options.quiet))

[mitsuhiko]

Ich hab noch eine generelle Anmekrung zur Namenswahl. Warum heißt es überhaupt CGIServer in colubird?
Für mich ist ein Apache mit CGI ehr ein CGI-Server

Ney. Eigentlich sollte es ja WSGIServer heißen, aber wenn zwei in einer Datei sind isses leicht komisch. Aber vielleicht bennen ichs wirklich um, bricht halt Kompatibilität.

Ich würde vorschlagen das ehr CGIbridge zu nennen. Denn es ist doch ehr eine Brücke zwischen dem ApacheCGIServer und der WSGI-Applikation. Oder vielleicht auch CGIAdapter...

Jup. Ein eigener Server isses ja so nicht, aber da wrapper in der Regel WSGIServer heißen wollte ich da anknüpfen.

Und was deinen Patch angeht, würde man die Serverstart message bei jedem Code Reload sehen. Weil da startet er den Prozess einfach neu.

[jens]

Ein eigener Server isses ja so nicht, aber da wrapper in der Regel WSGIServer heißen wollte ich da anknüpfen.

Dann finde ich die ursprüngliche Namenswahl nicht wirklich gelungen...

Und was deinen Patch angeht, würde man die Serverstart message bei jedem Code Reload sehen. Weil da startet er den Prozess einfach neu.

OK, dann müßte das irgendwo anders hin, aber generell ist eine Ausgabe IMHO ein muß!
Was ist mit der anderen Zeile???

[mitsuhiko]

Ein eigener Server isses ja so nicht, aber da wrapper in der Regel WSGIServer heißen wollte ich da anknüpfen.

Dann finde ich die ursprüngliche Namenswahl nicht wirklich gelungen...

Find ich nicht. Ist ja ein Diener

Und was deinen Patch angeht, würde man die Serverstart message bei jedem Code Reload sehen. Weil da startet er den Prozess einfach neu.

OK, dann müßte das irgendwo anders hin, aber generell ist eine Ausgabe IMHO ein muß!
Was ist mit der anderen Zeile???

Den fix mit dem hostname ANY werde ich einbauen.

[jens]

Ein eigener Server isses ja so nicht, aber da wrapper in der Regel WSGIServer heißen wollte ich da anknüpfen.

Dann finde ich die ursprüngliche Namenswahl nicht wirklich gelungen...

Find ich nicht. Ist ja ein Diener

Dann nenn es meinetwegen CGIbutler, CGIservant oder irgendwie anders, aber nicht CGIServer...

[jens]

Noch eine Anmerkung zur Namenswahl... Zwar ist in der neuen Colubrid kein CGI-Server mehr drin, aber dennoch...

Wie im IRC "besprochen" redet PEP-0333 von nicht von CGI-Serber sondern von CGI-Gateway, das IMHO sehr gut passt...

Es sieht also so aus:

WebApp -> CGI-Handler -> CGI-Gateway -> Apache

Also lasst und bitte immer von CGI-Gateware sprechen, ja?

(Im übrigen hab ich mal die http://wiki.python.de/Colubrid/Hello_World überarbeitet und noch ein paar Anmerkungen hinzugefügt...)