Run on a non-default port about papercups

[rberrelleza]

@reichert621 that was the issue. setting $PORT was overriding both the backend and the frontend's port at the same time. I forgot that that react's webserver uses PORT.

[reichert621]

hey @rberrelleza!

my ~guess is that the PORT env variable is used in both our frontend app (as our webpack dev server or whatever is used under the hood in create-react-app) as well as our server (with the prod config), which is why you're getting the error that "something is already running"

though hmm... I'm a little confused why the frontend is trying to run with the prod config anyway 🤔

if you tell me a little bit more about what you're trying to do, I'd be happy to help out more :)

[rberrelleza]

I'm trying to run papercups on as a container on our kubernetes cluster, and I want to expose it over port 8080.

With production, what I meant is that I'm trying to get the port using an env var, like you have in prod.secret.exs.

[reichert621]

@rberrelleza my cofounder probably has more expertise around things like this 😅 we can probably hash this out more quickly over slack if you'd like to come by! https://join.slack.com/t/papercups-io/shared_invite/zt-gfs0d269-dEHm3SYs_5KmFKQ9YhBzDw

[rberrelleza]

Will do! Feel free to close the issue and I'll follow up there.

[cheeseblubber]

Just noticed Something is already running on port 8080. do you have something runnning on the container? If we tried to run it on another port like 7070 just as a test I wonder if it would work.

[rberrelleza]

@cheeseblubber I tried, but got the same error. I think @reichert621 is right and the PORT env var is being used for both front and backend.

[reichert621]

if you changed this line https://github.com/papercups-io/papercups/blob/master/config/dev.exs#L19 to use the environment variable, then yes -- it'll use the same port for both :/

definitely something we can fix! just want to make sure this is the issue you're facing :)

(note that if you manually change that value above, you'll likely have to update this one too: https://github.com/papercups-io/papercups/blob/master/assets/package.json#L67)

[reichert621]

I forgot about that too 😅 thanks for flagging!