diff options
Diffstat (limited to 'scripts/send-error-report')
-rwxr-xr-x | scripts/send-error-report | 20 |
1 files changed, 14 insertions, 6 deletions
diff --git a/scripts/send-error-report b/scripts/send-error-report index 15b5e84911..cfbcaa52cb 100755 --- a/scripts/send-error-report +++ b/scripts/send-error-report @@ -6,6 +6,9 @@ # Copyright (C) 2013 Intel Corporation # Author: Andreea Proca <andreea.b.proca@intel.com> # Author: Michael Wood <michael.g.wood@intel.com> +# +# SPDX-License-Identifier: GPL-2.0-only +# import urllib.request, urllib.error import sys @@ -62,7 +65,7 @@ def edit_content(json_file_path): def prepare_data(args): # attempt to get the max_log_size from the server's settings - max_log_size = getPayloadLimit("http://"+args.server+"/ClientPost/JSON") + max_log_size = getPayloadLimit(args.protocol+args.server+"/ClientPost/JSON") if not os.path.isfile(args.error_file): log.error("No data file found.") @@ -88,7 +91,7 @@ def prepare_data(args): log.error("Name needs to be provided either via "+userfile+" or as an argument (-n).") sys.exit(1) - while len(args.name) <= 0 and len(args.name) < 50: + while len(args.name) <= 0 or len(args.name) > 50: print("\nName needs to be given and must not more than 50 characters.") args.name, args.email = ask_for_contactdetails() @@ -132,18 +135,18 @@ def send_data(data, args): headers={'Content-type': 'application/json', 'User-Agent': "send-error-report/"+version} if args.json: - url = "http://"+args.server+"/ClientPost/JSON/" + url = args.protocol+args.server+"/ClientPost/JSON/" else: - url = "http://"+args.server+"/ClientPost/" + url = args.protocol+args.server+"/ClientPost/" req = urllib.request.Request(url, data=data, headers=headers) try: response = urllib.request.urlopen(req) except urllib.error.HTTPError as e: - logging.error(e.reason) + logging.error(str(e)) sys.exit(1) - print(response.read()) + print(response.read().decode('utf-8')) if __name__ == '__main__': @@ -187,6 +190,11 @@ if __name__ == '__main__': help="Return the result in json format, silences all other output", action="store_true") + arg_parse.add_argument("--no-ssl", + help="Use http instead of https protocol", + dest="protocol", + action="store_const", const="http://", default="https://") + args = arg_parse.parse_args() |