commitfest.postgresql.org Specify thread msgid does not work for pgsql-bugs(at)lists(dot)postgresql(dot)org

On Mon, Oct 21, 2024 at 08:00:00AM +0800, jian he wrote:

see picture attached.

It does not seem like you have used an exact message-id. We have
cases in the CF app where a pgsql-bugs messages is used, like this
one:
https://commitfest.postgresql.org/50/5242/

And a quick attempt at using one of the pgsql-bugs messages also seems
to work here.
--
Michael

On 2024-Oct-21, jian he wrote:

hi.
https://commitfest.postgresql.org/50/new/#
won't work for messages from pgsql-bugs
<pgsql-bugs(at)lists(dot)postgresql(dot)org>.

recently, I try to register an entry for
/messages/by-id/tencent_BAB00BBF45A08D941DF8290D9533941BB807@qq.com
but cannot.

see picture attached.

You need to use the Message-Id only, without the URL part of it, so it
has to start with "tencent". Also, the %40 needs to be an @ sign. The
@ is converted to %40 because of URL-encoding, but that's not the real
message-id.

I guess it would be helpful if the commitfest app can read a URL to a
mailing list post and reverse-engineer the message-id from it ... maybe
you'd have to propose a patch to this:
https://git.postgresql.org/gitweb/?p=pgcommitfest2.git;a=blob;f=pgcommitfest/commitfest/views.py;h=2c21cd3d623d86a6fc8a0f639ab127ece520eacf;hb=refs/heads/master#l387

--
Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/

On Mon, 21 Oct 2024 at 11:49, Alvaro Herrera <alvherre@alvh.no-ip.org> wrote:

I guess it would be helpful if the commitfest app can read a URL to a
mailing list post and reverse-engineer the message-id from it ... maybe
you'd have to propose a patch to this:
https://git.postgresql.org/gitweb/?p=pgcommitfest2.git;a=blob;f=pgcommitfest/commitfest/views.py;h=2c21cd3d623d86a6fc8a0f639ab127ece520eacf;hb=refs/heads/master#l387

I agree that would be helpful. Feel free to send a patch to me over
email, or create a PR on this repo:
https://github.com/JelteF/commitfest