From ccbc99fe4fa415a7d30ab173a3d5d8733cd50ef8 Mon Sep 17 00:00:00 2001 From: James Cammarata Date: Mon, 6 Jan 2014 10:40:47 -0500 Subject: [PATCH] Fixed splitting of role/user name when username has a '.' in it This may still be an issue if users create roles with a '.' in the name though. We will probably have to disallow that in the role naming convention. --- bin/ansible-galaxy | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/bin/ansible-galaxy b/bin/ansible-galaxy index a6845fc86e0..7fbc4049212 100755 --- a/bin/ansible-galaxy +++ b/bin/ansible-galaxy @@ -241,7 +241,10 @@ def api_lookup_role_by_name(api_server, role_name): role_name = urllib.quote(role_name) try: - user_name,role_name = role_name.split(".", 1) + parts = role_name.split(".") + user_name = ".".join(parts[0:-1]) + role_name = parts[-1] + print " downloading role '%s', owned by %s" % (role_name, user_name) except: print "Invalid role name (%s). You must specify username.rolename" % role_name sys.exit(1)