passmarked / seo

Rules related to checking for any SEO issues on the page given
https://passmarked.com/library/seo
Apache License 2.0
11 stars 11 forks source link

Added the rule for trailing slashes #9

Closed Johanndutoit closed 7 years ago

Johanndutoit commented 7 years ago

As requested per #5, the check for trailing slashes on URL's.

Had to put some thought into this and not quite happy but it's a good start for a discussion.

The rule currently assumes that only one version of this page should work, either with or without the trailing slash. So if /about/ and /about works, the notice is shown (and vice versa).

Let me know what everyone thinks. Ping @BarryBotha from the #5 for feedback too

BarryBotha commented 7 years ago

If we can check whether /path and /path/ returns the same content it will be better. You may have both urls - they shouldn't return the same content though

Johanndutoit commented 7 years ago

Agreed, have a idea on how to check that too. Just need run a few local checks

Johanndutoit commented 7 years ago

Just added the diff check for us, along with the rule key in the README