The regex should end on .* to allow for URLs where tbm=isch is the last parameter.
Also, the question mark starting the query string should be escaped, otherwise you're just saying "it can be either '/search' or '/searc'".
It usually still works because of the .+ immediately after, but it is terrible for readability and probably (hopefully) not what you intended.
The forward slash in the exclusive group should also be escaped.
Here you go: /https?:\/\/(www\.)?google\.[^\/]+\/search\?.+tbm=isch.*/
The regex should end on
.*to allow for URLs wheretbm=ischis the last parameter.Also, the question mark starting the query string should be escaped, otherwise you're just saying "it can be either '/search' or '/searc'".
It usually still works because of the
.+immediately after, but it is terrible for readability and probably (hopefully) not what you intended.The forward slash in the exclusive group should also be escaped.
Here you go:
/https?:\/\/(www\.)?google\.[^\/]+\/search\?.+tbm=isch.*/