API icon indicating copy to clipboard operation
API copied to clipboard

Rename config for docker

Open ionphractal opened this issue 7 years ago • 6 comments

After changing the domains to xip.io, a local Docker development environment was unable to load images (failing with 401 unauthorized). That is because the JWT token audience uri was still set to localhost and hence the token was not recognized as valid.

The configuration package loads the config from NODE_ENV. And as there was no development config and Docker uses "development" I though I'd rename the config and extend it with the required parameters.

Does this break anything on your side? @appinteractive @roschaefer

ionphractal avatar Oct 20 '18 15:10 ionphractal

This would break the local environment without docker. You might want to use development-docker.json as a filename instead.

appinteractive avatar Oct 21 '18 15:10 appinteractive

Hm so we might want to treat NODE_ENV=development-docker as development in code, right?

ionphractal avatar Oct 21 '18 16:10 ionphractal

@ionphractal great debugging! Could you use config/default-docker.json instead, please? We set NODE_APP_INSTANCE=docker in the Dockerfile. See the node-config wiki to learn how config files are loaded.

Apart from that :+1:

roschaefer avatar Nov 12 '18 17:11 roschaefer

@appinteractive I restarted the build once, can you tell me what's wrong with codacy right now?

roschaefer avatar Nov 30 '18 18:11 roschaefer

@roschaefer codacy?

appinteractive avatar Nov 30 '18 18:11 appinteractive

It's failing the build.

roschaefer avatar Nov 30 '18 19:11 roschaefer