#5741: dont disallow double percent signs in SafeConfigParser.set() keys.
diff --git a/Lib/ConfigParser.py b/Lib/ConfigParser.py
index 1861b5a..2fa323b 100644
--- a/Lib/ConfigParser.py
+++ b/Lib/ConfigParser.py
@@ -620,7 +620,6 @@
return ''.join(L)
_interpvar_re = re.compile(r"%\(([^)]+)\)s")
- _badpercent_re = re.compile(r"%[^%]|%$")
def _interpolate_some(self, option, accum, rest, section, map, depth):
if depth > MAX_INTERPOLATION_DEPTH:
@@ -667,9 +666,10 @@
# check for bad percent signs:
# first, replace all "good" interpolations
tmp_value = self._interpvar_re.sub('', value)
+ tmp_value = tmp_value.replace('%%', '')
# then, check if there's a lone percent sign left
- m = self._badpercent_re.search(tmp_value)
- if m:
+ percent_index = tmp_value.find('%')
+ if percent_index != -1:
raise ValueError("invalid interpolation syntax in %r at "
- "position %d" % (value, m.start()))
+ "position %d" % (value, percent_index))
ConfigParser.set(self, section, option, value)