-
Notifications
You must be signed in to change notification settings - Fork 139
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
API mechanism for reporting validity errors #811
Comments
With regards to adding properties on the URL value: what happens if you mutate the value using one of the property setters? For instance, let's imagine the path contains a % sign which is not part of a percent-encoded byte (e.g. Now let's imagine somebody uses the path setter to change the path to something valid. Does In general, how do we know there is not some other component which also contains non-valid contents (e.g. the same |
For options |
It would also reflect the last operation, wouldn't it? For instance, non-fatal validation errors can occur when an IPv4 address contains non-decimal parts - e.g. In other words, this property would not be idempotent.
I think this would be extremely difficult for users to understand or use effectively. This suggests that validity may not in fact be a property of the URL, but rather a property of the inputs used to create the URL. So C (including "strict" versions of property setters, etc) would be the best API, IMO. |
I think we need to start with flushing this out more. See also 1 of https://whatwg.org/faq#adding-new-features. |
What is the issue with the URL Standard?
The spec specifically calls out validity errors while parsing and recommends that implementations report those in some way. The URL API does not surface those in any way and leaves it up to the host implementation to report validity errors out of band.
It would likely be useful for the API to provide some reasonable means of handling validity state...
For instance, a few approaches come to mind:
A. Adding an
valid
orinvalid
property toURL
. The value would be boolean. It would not report the specific validation error but would simply indicate whether or not a validation error exists.B. Adding a
validationErrors
property that is an array of validation errors. These can be numeric codes or strings that represent the specific collection of validation errors defined in the spec. Anull
value for this property would indicate no validation errors. If a value is provided it must be an array. This would allow both simple checking for valid/invalid and specific reporting for individual validation errors.C. Adding an opt-in strict parsing mode that converts validation errors into thrown exceptions.
Of these options, I generally prefer
A
as I do not really think there's much practical reason to break down exactly why the URL is invalid as much as simply calling it out.The text was updated successfully, but these errors were encountered: