gitpython-developers
diff --git a/‎git/objects/submodule/base.py‎
Lines changed: 1 addition & 1 deletion b/‎git/objects/submodule/base.py‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎git/remote.py‎
Lines changed: 1 addition & 1 deletion b/‎git/remote.py‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎test/test_remote.py‎
Lines changed: 9 additions & 0 deletions b/‎test/test_remote.py‎
Lines changed: 9 additions & 0 deletions
diff --git a/‎test/test_submodule.py‎
Lines changed: 14 additions & 0 deletions b/‎test/test_submodule.py‎
Lines changed: 14 additions & 0 deletions
@@ -608,7 +608,7 @@ def add(
         # END verify url
 
         ## See #525 for ensuring git URLs in config-files are valid under Windows.
-        url = Git.polish_url(url)
+        url = Git.polish_url(url, expand_vars=False)
 
         # It's important to add the URL to the parent config, to let `git submodule` know.
         # Otherwise there is a '-' character in front of the submodule listing:
 
@@ -808,7 +808,7 @@ def create(cls, repo: "Repo", name: str, url: str, allow_unsafe_protocols: bool
         """
         scmd = "add"
         kwargs["insert_kwargs_after"] = scmd
-        url = Git.polish_url(url)
+        url = Git.polish_url(url, expand_vars=False)
         if not allow_unsafe_protocols:
             Git.check_unsafe_protocols(url)
         repo.git.remote(scmd, "--", name, url, **kwargs)
 
@@ -4,6 +4,7 @@
 # 3-Clause BSD License: https://opensource.org/license/bsd-3-clause/
 
 import gc
+import os
 import os.path as osp
 from pathlib import Path
 import random
@@ -685,6 +686,14 @@ def test_multiple_urls(self, rw_repo):
         # Will raise fatal: Will not delete all non-push URLs.
         self.assertRaises(GitCommandError, remote.delete_url, test3)
 
+    @with_rw_repo("HEAD", bare=False)
+    def test_create_does_not_expand_environment_variables_in_url(self, rw_repo):
+        url = "https://example.com/${GITPYTHON_TEST_SECRET}/repo.git"
+        with mock.patch.dict(os.environ, {"GITPYTHON_TEST_SECRET": "sensitive-value"}):
+            remote = rw_repo.create_remote("untrusted", url)
+
+        assert remote.url == url
+
     def test_fetch_error(self):
         rem = self.rorepo.remote("origin")
         msg = (
 
@@ -1355,6 +1355,20 @@ def test_add_no_clone_multi_options_argument(self, rwdir):
         with self.assertRaises(cp.NoOptionError):
             sm_config.get_value("core", "eol")
 
+    @with_rw_directory
+    def test_add_does_not_expand_environment_variables_in_url(self, rwdir):
+        parent = git.Repo.init(osp.join(rwdir, "parent"))
+        url = osp.join(rwdir, "${GITPYTHON_TEST_SECRET}")
+        source = git.Repo.init(url)
+        Path(source.working_tree_dir, "file").write_text("content")
+        source.index.add(["file"])
+        source.index.commit("initial")
+
+        with mock.patch.dict(os.environ, {"GITPYTHON_TEST_SECRET": "sensitive-value"}):
+            sm = Submodule.add(parent, "new", "new", url)
+
+        assert sm.url == Git.polish_url(url, expand_vars=False)
+
     @with_rw_repo("HEAD")
     def test_submodule_add_unsafe_url(self, rw_repo):
         with tempfile.TemporaryDirectory() as tdir: