<p></p>
<p dir="auto">So I'm happy to merge this, but I would like to revisit some of the conversation in <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="171028018" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/1267" data-hovercard-type="pull_request" data-hovercard-url="/openstreetmap/openstreetmap-website/pull/1267/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/pull/1267">#1267</a> , namely</p>
<blockquote>
<p dir="auto">If we were happy to only match the domain before then why not stick with that but broaden the domain match? Why start trying to (badly) match the rest of the URL as well, which just means it will break every time google tweak their URLs...</p>
</blockquote>
<p dir="auto">The currently proposed regex is even more complex than the complex one we had already. So in my view we should choose between these:</p>
<ul dir="auto">
<li>If we're trying to stop everything from the google/googleapis domains, then we don't need the complexity in the path matching.</li>
<li>If we're trying to block the paths selectively, while permitting certain other paths to continue to work, then we need to add unit tests to a) illustrate which paths are supported and b) ensure these continue to work when the regex is next refactored.</li>
</ul>
<p dir="auto">I suspect we can just block all paths though, and therefore simplify the regex substantially. Thoughts?</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/3512#issuecomment-1090472903">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLKJH6M6EPJK7DMVGM3VDW4LZANCNFSM5RVXIWSQ">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLLNNE4NH4SUCDMVEGDVDW4LZA5CNFSM5RVXIWS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOID7UXRY.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/pull/3512/c1090472903</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/3512#issuecomment-1090472903",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/3512#issuecomment-1090472903",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>