RFR: 1001: CommentPosterWorkItem at openjdk/jdk#3425

Erik Joelsson erikj at openjdk.java.net
Tue Apr 27 22:46:16 UTC 2021


The mlbridge is currently spamming a PR with duplicated mail comments. This is caused by a regexp that fails to identify a comment as a bridged comment. The identification in this case is a Base64 encoded string that we generated. The regexp is missing + and / as valid characters in such a string. This change adds them and also adds a test, which includes the particular string that is currently causing the failure in production.

I need to get this deployed ASAP.

-------------

Commit messages:
 - Fix imports
 - Fix regexp for matching Base64 strings

Changes: https://git.openjdk.java.net/skara/pull/1139/files
 Webrev: https://webrevs.openjdk.java.net/?repo=skara&pr=1139&range=00
  Issue: https://bugs.openjdk.java.net/browse/SKARA-1001
  Stats: 22 lines in 2 files changed: 21 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/skara/pull/1139.diff
  Fetch: git fetch https://git.openjdk.java.net/skara pull/1139/head:pull/1139

PR: https://git.openjdk.java.net/skara/pull/1139


More information about the skara-dev mailing list