[wp-meta] [Making WordPress.org] #4322: Invalid length search inputs should be handled more cleanly

[Making WordPress.org]

#4322: Invalid length search inputs should be handled more cleanly
----------------------------+---------------------------
 Reporter:  jonoaldersonwp  |      Owner:  (none)
     Type:  defect          |     Status:  new
 Priority:  lowest          |  Milestone:
Component:  General         |   Keywords:  seo analytics
----------------------------+---------------------------
 Very long search requests, such as
 [https://wordpress.org/plugins/search/catszzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzLorem%20ipsum%20dolor%20sit%20amet,%20consectetur%20adipiscing%20elit.%20Mauris%20pulvinar%20eget%20tellus%20vel%20faucibus.%20Maecenas%20eu%20ullamcorper%20est,%20a%20mattis%20quam.%20Aliquam%20venenatis%20massa%20et%20sem%20ornare%20porta.%20Fusce%20diam%20dolor,%20faucibus%20id%20lorem%20eu,%20viverra%20suscipit%20massa.%20Maecenas%20a%20magna%20vehicula,%20auctor%20est%20vel,%20eleifend%20libero.%20Nam%20pulvinar%20eget%20metus%20eu%20volutpat.%20Vestibulum%20hendrerit%20porta%20ante,%20nec%20dapibus%20nibh%20pretium%20at.%20Phasellus%20semper%20aliquet%20sapien,%20id%20pretium%20felis%20rhoncus%20nec.%20Donec%20sed%20libero%20nec%20sapien%20finibus%20aliquam%20eget%20a%20nulla.%20Cras%20posuere%20nisl%20ut%20felis%20commodo,%20convallis%20tincidunt%20dolor%20accumsan.%20Aenean%20accumsan,%20nisi%20et%20auctor%20fermentum,%20odio%20orci%20ultricies%20lorem,%20at%20lobortis%20enim%20velit%20a%20
 metus.%20Curabitur%20laoreet%20porta%20venenatis.%20Morbi%20convallis%20rutrum%20nunc,%20et%20aliquam%20est%20luctus%20non.%20Aliquam%20erat%20volutpat.%20Praesent%20vitae%20ipsum%20orci.%20Sed%20dictum%20ex%20augue,%20tempor%20consequat%20sem%20volutpat%20at.%20Praesent%20suscipit%20mollis%20suscipit.%20Maecenas%20vitae%20elit%20in%20diam%20euismod%20vulputate%20id%20a%20ante.%20In%20placerat%20ac%20erat%20sed%20finibus.%20Sed%20gravida%20est%20nec%20ex%20maximus,%20sed%20placerat%20diam%20varius.%20Vestibulum%20ut%20lorem%20sollicitudin,%20viverra%20velit%20in,%20rutrum%20velit.%20Aliquam%20malesuada%20quis%20sem%20et%20feugiat.%20Maecenas%20pulvinar%20gravida%20laoreet.%20Etiam%20diam%20mi,%20ultrices%20ac%20nunc%20luctus,%20euismod%20eleifend%20neque.%20Ut%20neque%20libero,%20faucibus%20at%20sollicitudin%20at,%20iaculis%20non%20sem.%20Suspendisse%20sagittis%20tortor%20a%20leo%20interdum%20gravida.%20Integer%20at%20libero%20augue.%20Phasellus%20non%20quam%20feugiat,%20eleifend%20
 erat%20ac,%20accumsan%20quam.%20Nulla%20augue%20magna,%20lacinia%20et%20imperdiet%20eget,%20eleifend%20vel%20ligula.%20Duis%20porta%20tortor%20quis%20felis%20hendrerit%20aliquam.%20Nam%20vehicula%20interdum%20risus,%20non%20sagittis%20purus%20porta%20nec.%20Fusce%20aliquam%20augue%20at%20ligula%20luctus,%20nec%20volutpat%20nibh%20luctus.%20Etiam%20et%20nunc%20semper,%20elementum%20risus%20et,%20imperdiet%20metus.%20Mauris%20id%20justo%20cursus,%20lacinia%20eros%20nec,%20volutpat%20arcu.%20Etiam%20molestie%20dolor%20id%20commodo%20iaculis.%20Quisque%20suscipit%20libero%20ut%20neque%20ullamcorper,%20et%20mollis%20mi%20elementum.%20Sed%20congue%20eget%20lectus%20vel%20aliquet.%20Maecenas%20ultrices%20eros%20eget%20urna%20dapibus%20feugiat.%20Vivamus%20nisl%20nisl,%20varius%20vel%20congue%20vitae,%20venenatis%20at%20turpis.%20Morbi%20sapien%20sem,%20maximus%20eget%20sollicitudin%20eget,%20ullamcorper%20nec%20lectus.%20Sed%20vulputate%20leo%20vitae%20lorem%20ultrices%20porta.%20Ut%20inte
 rdum,%20turpis%20at%20suscipit%20volutpat,%20massa%20tellus%20consequat%20mi,%20at%20luctus%20velit%20dui%20a%20ex.%20Donec%20nec%20varius%20sem,%20nec%20vehicula%20urna.%20Sed%20erat%20felis,%20rhoncus%20ac%20molestie%20non,%20laoreet%20at%20dolor.%20In%20dolor%20est,%20condimentum%20at%20lectus%20quis,%20rhoncus%20commodo%20velit.%20Donec%20tempor%20vitae%20mi%20quis%20pulvinar.%20Interdum%20et%20malesuada%20fames%20ac%20ante%20ipsum%20primis%20in%20faucibus.%20Ut%20turpis%20lectus,%20sagittis%20nec%20dapibus%20in,%20facilisis%20et%20ante.%20Donec%20sollicitudin,%20dolor%20id%20volutpat%20ornare,%20lorem%20orci%20ullamcorper%20ipsum,%20ac%20dignissim%20neque%20nibh%20a%20massa.%20Donec%20finibus%20scelerisque%20varius.%20Sed%20vitae%20erat%20a%20urna%20blandit%20volutpat.%20Vestibulum%20mattis,%20risus%20at%20cursus%20ullamcorper,%20ligula%20augue%20tempor%20erat,%20sed%20sagittis%20augue%20leo%20sed%20lectus.%20Curabitur%20eget%20erat%20urna.%20In%20a%20bibendum%20est,%20ac%20sol
 licitudin%20arcu.zzzzzzz%22%3E%3Ch2%3Ecat/
 this one] currently outputs malformed canonicals, hreflang values, rel
 next vals, etc.

 Specifically, the values for these either output an additional trailing
 slash (e.g., `https://wordpress.org/plugins/search//`), or, output a chunk
 of the raw URL.

 In both cases, the value for these tags should be
 `https://wordpress.org/plugins/search/`.

-- 
Ticket URL: <https://meta.trac.wordpress.org/ticket/4322>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org