search

eWaterCycle / grpc4bmi

gRPC wrapper for model with a Basic modeling interface
https://grpc4bmi.readthedocs.io
Apache License 2.0
5 stars 4 forks source link

Bind to localhost instead of 0.0.0.0 #59

Open sverhoeven opened 4 years ago

sverhoeven commented 4 years ago

The BmiClientDocker and BmiClientSingularity now create Docker and Singularity containers with a random port bounded to 0.0.0.0 . This seems too much permission because client and server are located on the same machine so binding to localhost and 127.0.0.1 should be enough.

Could an extra argument be added to the constructor with host='localhost' of the BimClient* classes?

goord commented 4 years ago

You mean make localhost everywhere the default host?