Dokku app names are cleaned up

#3
#5

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 - nginx / client_max_body_size option - set by command line option or environmental variable.
 - nginx / proxy-read-timeout option - set by command line option or environmental variable.
+- Dokku app names are cleaned up. Invalid characters are changed to "-". Lower case is enforced.
 
 ## Changed
 

dokkusd/util.py

@@ -18,6 +18,14 @@ def get_remote_names_configured(git_remote_verbose_output):
     return out
 
 
+def clean_dokku_app_name(app_name: str) -> str:
+    app_name = app_name.replace("_", "-")
+    app_name = app_name.replace(":", "-")
+    app_name = app_name.replace("/", "-")
+    app_name = app_name.replace("\\", "-")
+    return app_name.lower()
+
+
 class Task:
     def __init__(
         self,
@@ -31,7 +39,7 @@ def __init__(
         self.remote_user = remote_user
         self.remote_host = remote_host
         self.remote_port = remote_port
-        self.app_name = app_name
+        self.app_name = clean_dokku_app_name(app_name)
 
     def _dokku_command(self, command):
         full_command = [

test/test_clean_dokku_app_name.py

@@ -0,0 +1,25 @@
+from dokkusd.util import clean_dokku_app_name
+
+
+def test_fine_1():
+    assert "cat" == clean_dokku_app_name("cat")
+
+
+def test_bad_1():
+    assert "cat-" == clean_dokku_app_name("cat_")
+
+
+def test_bad_2():
+    assert "cat-" == clean_dokku_app_name("cat:")
+
+
+def test_bad_3():
+    assert "cat-" == clean_dokku_app_name("cat/")
+
+
+def test_bad_4():
+    assert "cat-" == clean_dokku_app_name("cat\\")
+
+
+def test_bad_5():
+    assert "cat" == clean_dokku_app_name("CAT")