bpo-XXXXX: Exclude symlinks from zoneinfo.available_timezones() results\n\nSometimes available_timezones() was including 'localtime' in its results\nbecause it wasn't properly handling symlinks. This change ensures that\nsymlinks like 'localtime' are excluded from the available timezones list,\nproviding consistent results across different systems.\n\nAdded a test case to verify that symlinks are properly excluded.

Author: MengAiDev

Lib/test/test_zoneinfo/test_zoneinfo.py

@@ -1941,6 +1941,27 @@ def test_exclude_posixrules(self):
                 actual = self.module.available_timezones()
                 self.assertEqual(actual, expected)
 
+    def test_exclude_symlinks(self):
+        expected = {
+            "America/New_York",
+            "Europe/London",
+        }
+
+        tree = list(expected) + ["localtime"]
+
+        with tempfile.TemporaryDirectory() as td:
+            # Create regular timezone files
+            for key in expected:
+                self.touch_zone(key, td)
+            
+            # Create a symlink named "localtime" pointing to one of the timezone files
+            os.symlink("America/New_York", os.path.join(td, "localtime"))
+
+            with self.tzpath_context([td]):
+                actual = self.module.available_timezones()
+                self.assertEqual(actual, expected)
+                self.assertNotIn("localtime", actual)
+
 
 class CTestModule(TestModule):
     module = c_zoneinfo

Lib/zoneinfo/_tzpath.py

@@ -155,6 +155,10 @@ def valid_key(fpath):
 
             for file in files:
                 fpath = os.path.join(root, file)
+                
+                # Skip symlinks to avoid including files like 'localtime'
+                if os.path.islink(fpath):
+                    continue
 
                 key = os.path.relpath(fpath, start=tz_root)
                 if os.sep != "/":  # pragma: nocover