Merge pull request #244 from kinostl/bugfix/multi-domain

Add multi-domain workaround due to ClickCommand.

docs/DESIGN.md

@@ -16,11 +16,12 @@ An app is simply a `git` repository with some additional files on the top level,
 
 ### `Procfile` format
 
-`piku` recognizes five kinds of process declarations in the `Procfile`:
+`piku` recognizes six kinds of process declarations in the `Procfile`:
 
 * `wsgi` workers, in the format `dotted.module:entry_point` (Python-only)
 * `web` workers, which can be anything that honors the `PORT` environment variable
 * `static` workers, which simply mount the first argument as the root static path
+* `release` which is a special worker that is run once when the app is deployed, after installing deps (can be useful for build steps).
 * `cron` workers, which require a simplified `cron` expression preceding the command to be run (e.g. `cron: */5 * * * * python batch.py` to run a batch every 5 minutes)
 * `worker` processes, which are standalone workers and can have arbitrary names
 
@@ -32,6 +33,7 @@ worker: python long_running_script.py
 fetcher: python fetcher.py
 # Simple cron expression: minute [0-59], hour [0-23], day [0-31], month [1-12], weekday [1-7] (starting Monday, no ranges allowed on any field)
 cron: 0 0 * * * python midnight_cleanup.py
+release: python initial_cleanup.py
 ```
 
 ...whereas a generic app would be:

examples/python-postgres/requirements.txt

@@ -1,4 +1,4 @@
 Django>=2.2.9
 pytz==2019.1
-sqlparse==0.3.0
+sqlparse==0.4.4
 psycopg2-binary==2.8.3

piku.py

@@ -727,6 +727,10 @@ def spawn_app(app, deltas={}):
 
         # Set up nginx if we have NGINX_SERVER_NAME set
         if 'NGINX_SERVER_NAME' in env:
+            # Hack to get around ClickCommand
+            env['NGINX_SERVER_NAME'] = env['NGINX_SERVER_NAME'].split(',')
+            env['NGINX_SERVER_NAME'] = ' '.join(env['NGINX_SERVER_NAME'])
+
             nginx = command_output("nginx -V")
             nginx_ssl = "443 ssl"
             if "--with-http_v2_module" in nginx: