Skip to content

Commit

Permalink
Enable issue escaping
Browse files Browse the repository at this point in the history
  • Loading branch information
Christoph-Koschel committed Jul 8, 2023
1 parent c90d4c3 commit 85df7d0
Show file tree
Hide file tree
Showing 2 changed files with 29 additions and 2 deletions.
6 changes: 5 additions & 1 deletion action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -75,4 +75,8 @@ inputs:
GITHUB_URL:
description: 'Base url of GitHub API'
required: false
default: ${{ github.api_url }}
default: ${{ github.api_url }}
ESCAPE:
description: 'Escape all special Markdown characters'
required: false
default: true
25 changes: 24 additions & 1 deletion main.py
Original file line number Diff line number Diff line change
Expand Up @@ -295,6 +295,9 @@ class TodoParser(object):
ORG_PROJECTS_PATTERN = re.compile(r'(?<=org projects:\s).+')

def __init__(self):
# Determine if the Issues should be escaped.
self.should_escape = os.getenv('INPUT_ESCAPE', 'true') == 'true'

# Load any custom identifiers, otherwise use the default.
custom_identifiers = os.getenv('INPUT_IDENTIFIERS')
self.identifiers = ['TODO']
Expand Down Expand Up @@ -530,7 +533,10 @@ def _extract_issue_if_exists(self, comment, marker, code_block):
elif org_projects:
issue.org_projects.extend(org_projects)
elif len(cleaned_line):
issue.body.append(cleaned_line)
if self.should_escape:
issue.body.append(self._escape_markdown(cleaned_line))
else:
issue.body.append(cleaned_line)

if issue is not None and issue.identifier is not None and self.identifiers_dict is not None:
for identifier_dict in self.identifiers_dict:
Expand All @@ -541,6 +547,23 @@ def _extract_issue_if_exists(self, comment, marker, code_block):

return issue

@staticmethod
def _escape_markdown(comment):
# All basic characters according to: https://www.markdownguide.org/basic-syntax
must_escaped = ['\\', '<', '>', '#', '`', '*', '_', '[', ']', '(', ')', '!', '+', '-', '.', '|', '{', '}', '~', '=']

escaped = ''

# Linear Escape Algorithm, because the algorithm ends in an infinite loop when using the function 'replace',
# which tries to replace all backslashes with duplicate backslashes, i.e. also the already other escaped
# characters.
for c in comment:
if c in must_escaped:
escaped += '\\' + c
else:
escaped += c
return escaped

def _get_line_status(self, comment):
"""Return a Tuple indicating whether this is an addition/deletion/unchanged, plus the cleaned comment."""
addition_search = self.ADDITION_PATTERN.search(comment)
Expand Down

0 comments on commit 85df7d0

Please sign in to comment.