Plugin URLs should 301 to lowercase

[jonoaldersonwp]

Plugin URLs like ​https://wordpress.org/plugins/Documentation/ should 301 to lowercase.

See ​https://wordpress.org/plugins/Documentation/?a=b&d=d

[dd32]

In 10267:

Plugin Directory: Redirect plugins to their canonical location, avoiding mixed case urls.

Fixes #5324.

[jonoaldersonwp]

Could this preserve URL parameters, please?

[dd32]

Which URL parameters?

[jonoaldersonwp]

Any/all. I.e., we should redirect the path to lowercase, rather than redirecting the whole request to the canonical.

[dd32]

Okay, there aren't any URL parameters that need keeping here then.

[jonoaldersonwp]

We rely on various UTM parameters for Google Analytics.

[dd32]

Thank you for answering my question.

I don't think we care for keeping the query vars anywhere else, I don't think I'm going to add them here. We don't use UTM links anywhere on WordPress.org, although some plugins and 3rd parties might be setting them.

In other words: I'm leaving this as-is. If 3rd parties set the correct URLs they'll pass through, but I'm not jumping through hoops for it here.

[jonoaldersonwp]

We don't need to add them in, we need to not remove them. That's not best practice, and it'll cause us headaches in the future if/when we start to mature our analytics ecosystem.

[jonoaldersonwp]

Looks like we're still stripping parameters.

[dd32]

Replying to jonoaldersonwp:

Looks like we're still stripping parameters.

We won't be keeping them, since we don't use them.

This has been as it is for 3 years, so I'm not inclined to say we need to reconsider this.

[jonoaldersonwp]

We don't use them today, but that doesn't mean that we won't tomorrow / in the future. That it's been broken for years isn't a good reason not to fix it :D

[dd32]

Fixing something that isn't broken isn't a good reason to keep it either.

If we ever end up using them, then we can add the functionality at that time.

Keeping this ticket open forever for something that benefits no-one currently benefits no-one in the long run. It only makes the open ticket count longer.

[dd32]

Retaining query stripping. I'm not investing time in support for a feature we have no intention of using.