trailingPeriod setting not honored on subdomains about url-regex-safe HOT 7 CLOSED

I experience the same problem. Are there any plans to resolve this?

from url-regex-safe.

PR welcome!

from url-regex-safe.

I worked around this by adapting the path part from:

(?:[/?#][^\s")']*)?

to:

(?:[/?#][^\s")']*[^\s")'.?!])?

which is probably not quite right but seemed to cover all the cases I threw at it

from url-regex-safe.

Can you submit a PR and maybe a test? @birtles would love to get this in

from url-regex-safe.

Can you submit a PR and maybe a test? @birtles would love to get this in

Sure, I've submitted #29 for this.

from url-regex-safe.

v4.0.0 released with this, thank you @birtles

release notes @ https://github.com/spamscanner/url-regex-safe/releases/tag/v4.0.0

note: this version now requires node v14+

from url-regex-safe.

Thank you! 🙏

from url-regex-safe.