[PATCH v3] Move to msgid based URLs

Konstantin Ryabitsev konstantin at linuxfoundation.org
Tue Nov 27 06:59:02 AEDT 2018


On Wed, Oct 31, 2018 at 01:06:56AM +1100, Daniel Axtens wrote:
>Migrate our URL schema as follows:
>
>Patches:       /project/<linkname>/patch/<msgid>/
>Cover Letters: /project/<linkname>/cover/<msgid>/
>
>The usual sub-resources (mbox, raw) hang off those URLs.
>The old style URLs (/patch/NNN/*, /cover/NNN/*) redirect appropriately.
>
>Also add /project/<linkname>/comment/<msgid>/ which redirects you
>in the same way /comment/NNN/ does.
>
>Add /message/<msgid> as well. This does not require a project,
>so if a mail has been sent to multiple projects, the project that you
>will be redirected to is arbitrary.
>
>This patch doesn't expose the new /message/ URL in the UI anywhere,
>we can address that in a follow-up.
>
>I also haven't attempted to do anything meaningful with series.
>
>Our database still stores message ids as with angle brackets; we
>just work around that rather than trying to migrate. That too can
>come later if we think the pain is justified.
>
>Partially-closes: #106
>Reported-by: Konstantin Ryabitsev <konstantin at linuxfoundation.org>
>Reported-by: Linus Torvalds <torvalds at linux-foundation.org>
>Reported-by: Stephen Finucane <stephen at that.guru>
>Signed-off-by: Daniel Axtens <dja at axtens.net>
>
>---
>v1->v2: Move to msgid by default
>        Add release note
>        Add tests
>
>Rebase: rebased on top of master. Tested bundles.
>I'll apply it in the next couple of days in the absence of objection.

I'm very anxious to see it in place, so please consider me a captive 
cheerleader.

I'd also be very excited to see a new release with this and the database 
performance patches included -- perhaps a 2.2? Performance on projects 
with a long list of paginated patches is still rather slow on MySQL.

Best,
-K
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.ozlabs.org/pipermail/patchwork/attachments/20181126/152a89c1/attachment.sig>


More information about the Patchwork mailing list