laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/41293?usp=email )
Change subject: osmo-smdpp: update commandline help and default port ......................................................................
osmo-smdpp: update commandline help and default port
osmo-smdpp has built-in TLS support for some time now. Let's update update the commandline help to be more concise.
Since the built-in SSL/TLS support is enabled by default, let's also update the default port from 8000 to 443.
Change-Id: Ib5a069a8612beb1a9716a7514b498ec70d141178 --- M osmo-smdpp.py 1 file changed, 3 insertions(+), 3 deletions(-)
Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
diff --git a/osmo-smdpp.py b/osmo-smdpp.py index ed5046d..d1d6fd7 100755 --- a/osmo-smdpp.py +++ b/osmo-smdpp.py @@ -861,10 +861,10 @@
def main(argv): parser = argparse.ArgumentParser() - parser.add_argument("-H", "--host", help="Host/IP to bind HTTP to", default="localhost") - parser.add_argument("-p", "--port", help="TCP port to bind HTTP to", default=8000) + parser.add_argument("-H", "--host", help="Host/IP to bind HTTP(S) to", default="localhost") + parser.add_argument("-p", "--port", help="TCP port to bind HTTP(S) to", default=443) parser.add_argument("-c", "--certdir", help=f"cert subdir relative to {DATA_DIR}", default="certs") - parser.add_argument("-s", "--nossl", help="do NOT use ssl", action='store_true', default=False) + parser.add_argument("-s", "--nossl", help="disable built in SSL/TLS support", action='store_true', default=False) parser.add_argument("-v", "--verbose", help="dump more raw info", action='store_true', default=False) parser.add_argument("-b", "--brainpool", help="Use Brainpool curves instead of NIST", action='store_true', default=False)
-
laforge