Fix url constraint application

[sydney-runkle]

Fix #10914

[codspeed-hq]

CodSpeed Performance Report

Merging #10922 will not alter performance

_{Comparing urls-str-constraints (21b2401) with main (22a9af2)}

Summary

✅ 46 untouched benchmarks

[github-actions]

Coverage report

This PR does not seem to contain any modification to coverable code.

[davidhewitt]

Seems correct with one simplification.

[Viicos]

Looks good, hopefully we'll soon be able to rework this annotation application because the function-wrap check is quite unfortunate

[antazoey]

I used to be able to use FileURL for AnyUrl in my models (I have a field that can literally be AnyUrl, including file ones)... Should I make a separate issue or does this also fix that? Is there a way we can have a test for that use-case?

[sydney-runkle]

Looks good, hopefully we'll soon be able to rework this annotation application because the function-wrap check is quite unfortunate

yes, very much so, I'd like to rework the url types more - I think we could still reconsider validation on init, or at least go about it in a different way

[cloudflare-workers-and-pages]

Deploying pydantic-docs with    Cloudflare Pages

Latest commit:	21b2401
Status:	✅  Deploy successful!
Preview URL:	https://4e7b8150.pydantic-docs.pages.dev
Branch Preview URL:	https://urls-str-constraints.pydantic-docs.pages.dev

View logs

[sydney-runkle]

@antazoey,

The issue is that AnyUrl requires a host, whereas FileUrl does not. So perhaps you can get the desired result with AnyUrl | FileUrl as your type?

[antazoey]

@antazoey,

The issue is that AnyUrl requires a host, whereas FileUrl does not. So perhaps you can get the desired result with AnyUrl | FileUrl as your type?

OK I can do that. Is host required now only on 2.10 or was it required before but just not failing validation?