Merge branch 'cors-error-uploading-issue-58' into 'main'

fix(upload): support manuscript upload for dev build on a server

Closes #58

See merge request !172

FAQ.md

@@ -55,3 +55,7 @@ x = await User.query().where({username:"0000000210481437"}).first()
 x.admin = true
 x.save()
 ```
+
+## What should PUBLIC_CLIENT_HOST be set to?
+
+This environment variable is currently only needed when deploying a _development_ build to a server (not to localhost). Set it to the server's name, e.g. `subdomain.myserver.com`.

config/custom-environment-variables.js

@@ -5,7 +5,7 @@ module.exports = {
   },
   'pubsweet-client': {
     protocol: 'CLIENT_PROTOCOL',
-    host: 'CLIENT_HOST',
+    host: 'PUBLIC_CLIENT_HOST',
     port: 'CLIENT_PORT',
   },
   'pubsweet-server': {

docker-compose.yml

@@ -22,6 +22,7 @@ services:
       - SERVER_PROTOCOL=http
       - SERVER_HOST=server
       - SERVER_PORT=${SERVER_PORT:-3000}
+      - PUBLIC_CLIENT_HOST=${PUBLIC_CLIENT_HOST:-0.0.0.0}
     volumes:
       - ./app:/home/node/app/app
       - ./config:/home/node/app/config

webpack/plugins.js

@@ -46,6 +46,7 @@ module.exports = (opts = {}) => {
       'CLIENT_PROTOCOL',
       'CLIENT_HOST',
       'CLIENT_PORT',
+      'PUBLIC_CLIENT_HOST',
     ]),
   )