Skip to content

the current script doesn't seem to fully cover all Google search domains #1

@Cloudiver

Description

@Cloudiver

hi,
thanks so much for creating this script. It's been really helpful for me.

I noticed that the current script doesn't seem to fully cover all Google search domains. Specifically, it can't recognize links with multi-part top-level domains like .google.com.hk or .google.co.jp, which are listed among Google's supported domains

To fix this, we suggest a small modification to the existing regular expression.

the original regex:

/^https?:\/\/www\.google\.[a-z]{2,3}\/.*/

maybe change to this improved version or others:

/^https?:\/\/www\.google\.([a-z]{2,3}\.?){1,}\/.*/

This updated expression should allow the script to more broadly match Google URLs that include multi-level domains.

Hope this helps!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions