Skip to content

Commit

Permalink
Editorial: only care about the empty string when beStrict is false
Browse files Browse the repository at this point in the history
VerifyDnsLength will take care of it otherwise. (See #497 for additional context.)
  • Loading branch information
annevk committed Dec 2, 2024
1 parent c3d173f commit acbcafd
Showing 1 changed file with 15 additions and 10 deletions.
25 changes: 15 additions & 10 deletions url.bs
Original file line number Diff line number Diff line change
Expand Up @@ -917,19 +917,24 @@ concepts.
<li><p>If <var>result</var> is a failure value, <a>domain-to-ASCII</a> <a>validation error</a>,
return failure.

<li><p>If <var>result</var> is the empty string, <a>domain-to-ASCII</a> <a>validation error</a>,
return failure.

<li>
<p>If <var>beStrict</var> is false and <var>result</var> contains a
<a>forbidden domain code point</a>, <a>domain-invalid-code-point</a> <a>validation error</a>,
return failure.
<p>If <var>beStrict</var> is false:

<ol>
<li><p>If <var>result</var> is the empty string, <a>domain-to-ASCII</a> <a>validation error</a>,
return failure.

<li>
<p>If <var>result</var> contains a <a>forbidden domain code point</a>,
<a>domain-invalid-code-point</a> <a>validation error</a>, return failure.

<p class=note>Due to web compatibility and compatibility with non-DNS-based systems the
<a>forbidden domain code points</a> are a subset of those disallowed when <i>UseSTD3ASCIIRules</i>
is true. See also <a href="https://github.com/whatwg/url/issues/397">issue #397</a>.
<p class=note>Due to web compatibility and compatibility with non-DNS-based systems the
<a>forbidden domain code points</a> are a subset of those disallowed when
<i>UseSTD3ASCIIRules</i> is true. See also
<a href="https://github.com/whatwg/url/issues/397">issue #397</a>.
</ol>

<li><p><a for=/>Assert</a>: <var>result</var> does not contain a
<li><p><a for=/>Assert</a>: <var>result</var> is not the empty string and does not contain a
<a>forbidden domain code point</a>.

<li><p>Return <var>result</var>.
Expand Down

0 comments on commit acbcafd

Please sign in to comment.