docker documentation for config files is no longer valid

[alastair]

The docker installation instructions (https://github.com/solid/node-solid-server#use-docker) have a step:

docker cp solid:/usr/src/app/config.json .

However by default this file doesn't exist, I get an error:

Error: No such container:path: solid:/usr/src/app/config.json

I was able to successfully create a config file using solid init and then copy it

perhaps these instructions need to be updated to cover some recent changes to the docker setup?

[bourgeoa]

Thanks for raising the issue. The problem comes from the introduction of a solid docker image, which uses a different home folder (opt/solid/) than the original dockerfile (usr/src/app/).

Something should be updated. @angelo-v what do you think : change the documentation or align docker image and dockerfile ?

• align docker image and dockerfile has a big draw back : both are used in production
• then change the documentation, may be the only feasible solution.

[angelo-v]

We should clarify in the documentation, that the docker image is configured via environment variables (as it is common with docker). The config.json should not be used at all

[alastair]

Thanks for the comment. This helpfully answers my questions in https://github.com/solid/node-solid-server/issues/1453 too

[TallTed]

@angelo-v wrote:

The config.json should not be used at all

That's in direct contradiction to the current docs! (scroll up one paragraph from the header to which this links)

Configuring Solid via the config file can be a concise and convenient method and is the generally recommended approach.

(Quite likely, this should be combined with #1453.)

[angelo-v]

You are referring to a paragraph that is not about docker usage. My statement that you quote is about docker usage only.

[TallTed]

@angelo-v -- Ah, sorry, my mis-read. :-(