tyler.durden
/
PyKMIP
mirror of https://github.com/OpenKMIP/PyKMIP.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fixes configuration settings override by default script values 

This change fixes a bug with parameter handling when invoking
the server's main routine. The default values provided for the
parameters would override any configuration file settings if
the main routine was used to run the server. To fix this, the
default values were removed, requiring the user to explicitly
specify parameter values if they want them to override the
configuration settings. The parameter doc strings have been
updated to match these changes.

Fixes #217
This commit is contained in:
Peter Hamilton 2016-12-01 11:41:57 -05:00
parent 5c50ec73fe
commit 38e7635fc7
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
kmip/services/server/server.py
View File

@ -367,12 +367,12 @@ def build_argument_parser():
"--hostname",
action="store",
type="str",
default="127.0.0.1",
default=None,
dest="hostname",
help=(
"The host address the server will be bound to. A string "
"representing either a hostname in Internet domain notation or "
"an IPv4 address. Defaults to '127.0.0.1'."
"an IPv4 address. Defaults to None."
),
)
parser.add_option(
@ -380,12 +380,12 @@ def build_argument_parser():
"--port",
action="store",
type="int",
default=5696,
default=None,
dest="port",
help=(
"The port number the server will be bound to. An integer "
"representing a port number. Recommended to be 5696 according to "
"the KMIP specification. Defaults to 5696."
"the KMIP specification. Defaults to None."
),
)
parser.add_option(
@ -429,11 +429,12 @@ def build_argument_parser():
"--auth_suite",
action="store",
type="str",
default="Basic",
default=None,
dest="auth_suite",
help=(
"A string representing the type of authentication suite to use "
"when establishing TLS connections. Defaults to 'Basic'."
"when establishing TLS connections. Options include 'Basic' and "
"'TLS1.2'. Defaults to None."
),
)
parser.add_option(