dunglas / go-urlpattern

92%

main: 92%

LAST BUILD ON BRANCH fix/tokenizer-error-swallow

branch: fix/tokenizer-error-swallow

CHANGE BRANCH

x

Reset

Sync Branches

• fix/tokenizer-error-swallow
• chore/bump-deps
• chore/lint-fixes
• chore/upstream-whatwg-url
• fix/canonicalize-hash-error
• fix/process-hash-strip-prefix
• fix/string-delimiters
• main
• perf/duplicate-name-map
• perf/scheme-lookups
• spec/conformance
• test/benchmarks

Build # 24663169211

Build Type

Pull #4

github

Committed by dunglas

Commit Message

fix: propagate tokenizing error on trailing backslash

The escape-at-end branch checked err == nil instead of err != nil,
which caused two failure modes:
- Strict policy: the non-nil error was dropped and the tokenizer's
  index was never advanced, producing an infinite loop.
- Lenient policy: the function returned (nil, nil) after a single
  invalid-char token instead of continuing to tokenize the rest.

Add a regression test that times out if strict tokenization loops.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Pull Request Pull Request #4: fix: propagate tokenizing error on trailing backslash

Coverage Stats

1 of 1 new or added line in 1 file covered. (100.0%)

1488 of 1624 relevant lines covered (91.63%)

1033.67 hits per line