nteract / commuter

🚎 Notebook sharing hub

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Use COMMUTER_PORT variable everywhere

groodt opened this issue · comments

At the moment there is inconsistency between the variables the frontend uses for knowledge about the server PORT and the server itself.

See:

const port = process.env.COMMUTER_PORT || 4000;

vs

config.port = process.env.PORT || process.env.COMMUTER_PORT || 4000;

I think we should only use COMMUTER_PORT or default to 4000 everywhere.

I'll raise a PR to fix this shortly.