From fae7f80dc11c4895becd076d418a091b730b855d Mon Sep 17 00:00:00 2001 From: Madison Swain-Bowden Date: Thu, 17 Aug 2023 11:44:39 -0700 Subject: [PATCH] Prevent < and > in commit titles from breaking dag-sync slack message (#2837) * Prevent < and > in commit titles from breaking dag-sync slack message * Use HTML entities rather than odd unicode characters --- dag-sync.sh | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/dag-sync.sh b/dag-sync.sh index 165c7cb29f7..24003419336 100755 --- a/dag-sync.sh +++ b/dag-sync.sh @@ -30,6 +30,10 @@ have_dag=$(git log -p -1 "$new" --pretty=format: --name-only | grep "catalog/dag # Pull out the subject from the new commit subject=$(git log -1 --format='%s') +# Swap the < & > characters for their HTML entities so they aren't +# interpreted as delimiters by Slack +subject=${subject//>/>} +subject=${subject//