From 2bfefe8fdc06a04ad2f0613067b1bc850f5aaca2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pavel=20K=C3=A1cha?= <ph@cesnet.cz>
Date: Wed, 19 Jul 2017 15:52:47 +0200
Subject: [PATCH] Distinct exit statuses of command-line errors (thx
bodik@cesnet.cz)
---
warden3/warden_server/warden_server.py | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/warden3/warden_server/warden_server.py b/warden3/warden_server/warden_server.py
index 4c20d03..865a88b 100755
--- a/warden3/warden_server/warden_server.py
+++ b/warden3/warden_server/warden_server.py
@@ -1536,23 +1536,23 @@ def modify_client(**kwargs):
kwargs["hostname"] = kwargs["hostname"].lower()
if not isValidHostname(kwargs["hostname"]):
print >>sys.stderr, "Invalid hostname \"%s\"." % kwargs["hostname"]
- return 254
+ return 253
if kwargs["requestor"] is not None and not isValidEmail(kwargs["requestor"]):
print >>sys.stderr, "Invalid requestor email \"%s\"." % kwargs["requestor"]
- return 254
+ return 252
if kwargs["id"] is not None and not isValidID(kwargs["id"]):
print >>sys.stderr, "Invalid id \"%s\"." % kwargs["id"]
- return 254
+ return 251
for c in server.handler.db.get_clients():
if kwargs["name"] is not None and kwargs["name"].lower()==c.name:
print >>sys.stderr, "Clash with existing name: %s" % str(c)
- return 254
+ return 250
if kwargs["secret"] is not None and kwargs["secret"]==c.secret:
print >>sys.stderr, "Clash with existing secret: %s" % str(c)
- return 254
+ return 249
newid = server.handler.db.add_modify_client(**kwargs)
--
GitLab