Fix HTML comment appearing in Azure Pull Request mail notifications (#…

…2521) * Fix HTML comment appearing in Azure Pull Request mail notifications * [MegaLinter] Apply linters fixes --------- Co-authored-by: nvuillam <nvuillam@users.noreply.github.com>
oxsecurity · Apr 3, 2023 · c1633dc · c1633dc
1 parent 432ccb7
commit c1633dc
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -16,6 +16,7 @@ Note: Can be used with `oxsecurity/megalinter@beta` in your GitHub Action mega-l
 
 - Core
   - Fix failure of AzureCommentReporter when there is no pull request found in ENV vars
+  - Fix HTML comment appearing in Azure Pull Request mail notifications
 
 - Documentation
 

diff --git a/megalinter/reporters/AzureCommentReporter.py b/megalinter/reporters/AzureCommentReporter.py
@@ -89,7 +89,9 @@ def produce_report(self):
             existing_thread_comment_id = None
             for existing_thread in existing_threads:
                 for comment in existing_thread.comments or []:
-                    if "<!-- MegaLinter Status Report -->" in (comment.content or ""):
+                    if "MegaLinter is graciously provided by" in (
+                        comment.content or ""
+                    ):
                         existing_thread_comment = existing_thread
                         existing_thread_comment_id = existing_thread.comments[0].id
                         existing_thread_id = existing_thread.id

diff --git a/megalinter/reporters/GitlabCommentReporter.py b/megalinter/reporters/GitlabCommentReporter.py
@@ -130,7 +130,7 @@ def produce_report(self):
 
             # Check if there is already a MegaLinter comment
             for comment in existing_comments:
-                if "<!-- MegaLinter Status Report -->" in comment.body:
+                if "MegaLinter is graciously provided by" in comment.body:
                     existing_comment = comment
 
             # Process comment

diff --git a/megalinter/utils_reporter.py b/megalinter/utils_reporter.py
@@ -149,7 +149,6 @@ def build_markdown_summary(reporter_self, action_run_url):
             + "(https://www.ox.security/wp-content/uploads/2022/06/"
             + "logo.svg?ref=megalinter_comment)](https://www.ox.security/?ref=megalinter)_"
         )
-    p_r_msg += os.linesep + "<!-- MegaLinter Status Report -->"
     logging.debug("\n" + p_r_msg)
     return p_r_msg