trampgeek / jobe

jobe is a server that runs small programming jobs in a variety of programming languages
MIT License
111 stars 80 forks source link

Testing instructions on README.md wrongly mention --port=4000 #72

Open jmr-ua-pt opened 7 months ago

jmr-ua-pt commented 7 months ago

On README.md you include the following instructions to test a Jobe server from a client machine:

python3 testsubmit.py --host='jobe.somehow.somewhere' --port=4000

However, as far as I can tell, the Jobe server is listening on port 80, right?

trampgeek commented 7 months ago

In the default configuration, running Jobe natively on a server, you're right that Jobe will be using Port 80. However, if you run jobe in a container (e.g. see jobeinabox) you will often map port 80 in the container to another port outside it. Or you may choose to configure Apache on the Jobe host to accept requests on a different port. testubmit.py needs to be able to test these alternative configurations.

However, I can see that the README is confusing on this - I probably updated it to describe how to test JobeInAbox containers with port 80 mapped to 4000. I'll tweak the wording.

Thanks for reporting.