RFR: 1001: CommentPosterWorkItem at openjdk/jdk#3425

Tim Bell tbell at openjdk.java.net
Tue Apr 27 22:50:28 UTC 2021


On Tue, 27 Apr 2021 22:42:47 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

> 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.

Looks good

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

Marked as reviewed by tbell (Reviewer).

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


More information about the skara-dev mailing list