Define strings outside of loop

main
Bob Mottram 2021-09-13 18:08:45 +01:00
parent 58f4840dd9
commit cadb306b66
1 changed files with 6 additions and 4 deletions

View File

@ -825,6 +825,12 @@ def dangerousMarkup(content: str, allowLocalNetworkAccess: bool) -> bool:
"""Returns true if the given content contains dangerous html markup """Returns true if the given content contains dangerous html markup
""" """
separators = (['<', '>'], ['&lt;', '&gt;']) separators = (['<', '>'], ['&lt;', '&gt;'])
invalidStrings = (
'script', 'noscript',
'canvas', 'style', 'abbr',
'frame', 'iframe', 'html', 'body',
'hr', 'allow-popups', 'allow-scripts'
)
for separatorStyle in separators: for separatorStyle in separators:
startChar = separatorStyle[0] startChar = separatorStyle[0]
endChar = separatorStyle[1] endChar = separatorStyle[1]
@ -836,10 +842,6 @@ def dangerousMarkup(content: str, allowLocalNetworkAccess: bool) -> bool:
invalidPartials = () invalidPartials = ()
if not allowLocalNetworkAccess: if not allowLocalNetworkAccess:
invalidPartials = getLocalNetworkAddresses() invalidPartials = getLocalNetworkAddresses()
invalidStrings = ('script', 'noscript',
'canvas', 'style', 'abbr',
'frame', 'iframe', 'html', 'body',
'hr', 'allow-popups', 'allow-scripts')
for markup in contentSections: for markup in contentSections:
if endChar not in markup: if endChar not in markup:
continue continue