fix: skip external URLs in withBase

[zouhangwithsweet]

Reason

I found that withBase will add path for all path, if I want to use some web resource. It's no need. So I add some path check for some url

More

I have no idea for that there is necessary to check all path which is been wrapped withBase.

[posva]

Maybe is worth for withBase to skip paths that start with http.

BTW, in a regex you can write c? instead of c{0,1}

[kiaking]

Yeah I also think withBase skipping URL starting with https?:// sounds a better idea 👀 I wouldn't think it will break anything, if anything, it's breaking URLs for people so 😅

[zouhangwithsweet]

Maybe is worth for withBase to skip paths that start with http.

BTW, in a regex you can write c? instead of c{0,1}

OK, sir

[zouhangwithsweet]

Yeah I also think withBase skipping URL starting with https?:// sounds a better idea 👀 I wouldn't think it will break anything, if anything, it's breaking URLs for people so 😅

I will make changes according to your suggestions. :)

[posva]

@kiaking I think we should move the RE in a shared file. I saw multiple occurrences of it across the codebase

[posva]

Thanks!