-
-
Notifications
You must be signed in to change notification settings - Fork 629
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fields.Url
does not accept file
URLs without host
#2249
Comments
sloria
changed the title
Jan 21, 2025
fields.Url
does not accept file
URLsfields.Url
does not accept file
URLs without host
sorry for the delay. i was able to repro the issue. will investigate when i have some time. PRs welcome in the meantime |
0xDEC0DE
pushed a commit
to 0xDEC0DE/marshmallow
that referenced
this issue
Jan 21, 2025
Requires a modicum of special handling due to hostnames being optional. Fixes: Issue marshmallow-code#2249
0xDEC0DE
added a commit
to 0xDEC0DE/marshmallow
that referenced
this issue
Jan 21, 2025
Requires a modicum of special handling due to hostnames being optional. Fixes: Issue marshmallow-code#2249
0xDEC0DE
added a commit
to 0xDEC0DE/marshmallow
that referenced
this issue
Jan 21, 2025
Requires a modicum of special handling due to hostnames being optional. Fixes: Issue marshmallow-code#2249
0xDEC0DE
added a commit
to 0xDEC0DE/marshmallow
that referenced
this issue
Jan 21, 2025
Requires a modicum of special handling due to hostnames being optional. Fixes: Issue marshmallow-code#2249
0xDEC0DE
added a commit
to 0xDEC0DE/marshmallow
that referenced
this issue
Jan 21, 2025
Requires a modicum of special handling due to hostnames being optional. Fixes: Issue marshmallow-code#2249
sloria
added a commit
that referenced
this issue
Jan 23, 2025
* fix: add `file` handling to URL fields Requires a modicum of special handling due to hostnames being optional. Fixes: Issue #2249 * Add more tests cases; update changelog --------- Co-authored-by: Steven Loria <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Steps to reproduce
Run this test case:
Expected result
Actual behavior
Workaround
This works:
...but the
hostname
portion is entirely optional withfile
URLs.The text was updated successfully, but these errors were encountered: