fast-uri 4.1.1 → 4.1.3
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.github/workflows/ci.yml +9 -9
- package/.github/workflows/lock-threads.yml +1 -1
- package/.github/workflows/ossf-scorecard.yml +45 -0
- package/README.md +1 -0
- package/SECURITY.md +173 -0
- package/benchmark/benchmark.mjs +71 -0
- package/index.js +236 -42
- package/lib/schemes.js +384 -7
- package/lib/utils.js +302 -88
- package/package.json +1 -1
- package/test/component-safe-serialization.test.js +105 -0
- package/test/equal.test.js +43 -3
- package/test/fixtures/uri-js-parse.json +24 -1
- package/test/fixtures/uri-js-serialize.json +23 -1
- package/test/ipv6-canonical.test.js +34 -0
- package/test/ipv6-validation.test.js +90 -0
- package/test/malformed-percent.test.js +77 -0
- package/test/malformed-urn.test.js +61 -0
- package/test/parse.test.js +145 -0
- package/test/reserved-path-normalization.test.js +109 -0
- package/test/scheme-validation.test.js +122 -0
- package/test/security-normalization.test.js +101 -0
- package/test/security-utf8-encoding.test.js +0 -0
- package/test/security.test.js +211 -3
- package/test/serialize.test.js +144 -0
- package/test/uri-js-compatibility.test.js +0 -3
- package/test/uri-js.test.js +3 -3
- package/test/urn-full-input.test.js +29 -0
- package/test/websocket-query-preservation.test.js +24 -0
- package/types/index.d.ts +4 -0
- package/types/index.tst.ts +16 -0
package/.github/workflows/ci.yml
CHANGED
|
@@ -30,11 +30,11 @@ jobs:
|
|
|
30
30
|
matrix:
|
|
31
31
|
node-version: ['20', '22', '24', '26']
|
|
32
32
|
steps:
|
|
33
|
-
- uses: actions/checkout@v7
|
|
33
|
+
- uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
|
|
34
34
|
with:
|
|
35
35
|
persist-credentials: false
|
|
36
36
|
|
|
37
|
-
- uses: actions/setup-node@
|
|
37
|
+
- uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
|
|
38
38
|
with:
|
|
39
39
|
node-version: ${{ matrix.node-version }}
|
|
40
40
|
cache: 'npm'
|
|
@@ -54,11 +54,11 @@ jobs:
|
|
|
54
54
|
matrix:
|
|
55
55
|
node-version: ['16', '18', '20', '22', '24', '26']
|
|
56
56
|
steps:
|
|
57
|
-
- uses: actions/checkout@v7
|
|
57
|
+
- uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
|
|
58
58
|
with:
|
|
59
59
|
persist-credentials: false
|
|
60
60
|
|
|
61
|
-
- uses: actions/setup-node@
|
|
61
|
+
- uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
|
|
62
62
|
with:
|
|
63
63
|
node-version: ${{ matrix.node-version }}
|
|
64
64
|
cache: 'npm'
|
|
@@ -77,11 +77,11 @@ jobs:
|
|
|
77
77
|
permissions:
|
|
78
78
|
contents: read
|
|
79
79
|
steps:
|
|
80
|
-
- uses: actions/checkout@v7
|
|
80
|
+
- uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
|
|
81
81
|
with:
|
|
82
82
|
persist-credentials: false
|
|
83
83
|
|
|
84
|
-
- uses: actions/setup-node@
|
|
84
|
+
- uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
|
|
85
85
|
with:
|
|
86
86
|
node-version: '10'
|
|
87
87
|
cache: 'npm'
|
|
@@ -119,11 +119,11 @@ jobs:
|
|
|
119
119
|
permissions:
|
|
120
120
|
contents: read
|
|
121
121
|
steps:
|
|
122
|
-
- uses: actions/checkout@v7
|
|
122
|
+
- uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
|
|
123
123
|
with:
|
|
124
124
|
persist-credentials: false
|
|
125
125
|
|
|
126
|
-
- uses: actions/setup-node@
|
|
126
|
+
- uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
|
|
127
127
|
with:
|
|
128
128
|
node-version: '24'
|
|
129
129
|
cache: 'npm'
|
|
@@ -147,7 +147,7 @@ jobs:
|
|
|
147
147
|
permissions:
|
|
148
148
|
contents: write
|
|
149
149
|
pull-requests: write
|
|
150
|
-
uses: fastify/workflows/.github/workflows/plugins-ci.yml@
|
|
150
|
+
uses: fastify/workflows/.github/workflows/plugins-ci.yml@ef591e2186785d5ab36b9fe6a79c7ce2f1d94e57 # v7.0.0
|
|
151
151
|
with:
|
|
152
152
|
license-check: true
|
|
153
153
|
lint: true
|
|
@@ -0,0 +1,45 @@
|
|
|
1
|
+
name: OSSF Scorecard Analysis
|
|
2
|
+
|
|
3
|
+
# **What it does**: Runs OSSF Scorecard analysis on the repository and uploads the results.
|
|
4
|
+
# **Why we have it**: Security scanning.
|
|
5
|
+
|
|
6
|
+
on:
|
|
7
|
+
branch_protection_rule:
|
|
8
|
+
push:
|
|
9
|
+
branches:
|
|
10
|
+
- main
|
|
11
|
+
paths-ignore:
|
|
12
|
+
- "docs/**"
|
|
13
|
+
- "*.md"
|
|
14
|
+
schedule:
|
|
15
|
+
# ┌───────────── minute (0 - 59)
|
|
16
|
+
# │ ┌───────────── hour (0 - 23)
|
|
17
|
+
# │ │ ┌───────────── day of the month (1 - 31)
|
|
18
|
+
# │ │ │ ┌───────────── month (1 - 12 or JAN-DEC)
|
|
19
|
+
# │ │ │ │ ┌───────────── day of the week (0 - 6 or SUN-SAT)
|
|
20
|
+
# │ │ │ │ │
|
|
21
|
+
# │ │ │ │ │
|
|
22
|
+
# │ │ │ │ │
|
|
23
|
+
# * * * * *
|
|
24
|
+
- cron: "21 17 * * 0"
|
|
25
|
+
# Allows this workflow to be run manually from the Actions tab
|
|
26
|
+
workflow_dispatch:
|
|
27
|
+
|
|
28
|
+
# This allows a subsequently queued workflow run to interrupt previous runs
|
|
29
|
+
concurrency:
|
|
30
|
+
group: "${{ github.workflow }}-${{ github.event.pull_request.head.label || github.head_ref || github.ref }}"
|
|
31
|
+
cancel-in-progress: true
|
|
32
|
+
|
|
33
|
+
permissions:
|
|
34
|
+
contents: read
|
|
35
|
+
|
|
36
|
+
jobs:
|
|
37
|
+
analysis:
|
|
38
|
+
name: OSSF Scorecard Analysis
|
|
39
|
+
permissions:
|
|
40
|
+
contents: read
|
|
41
|
+
id-token: write
|
|
42
|
+
security-events: write
|
|
43
|
+
uses: fastify/workflows/.github/workflows/reusable-ossf-scorecard.yml@ef591e2186785d5ab36b9fe6a79c7ce2f1d94e57 #v7.0.0
|
|
44
|
+
with:
|
|
45
|
+
publish_results: true
|
package/README.md
CHANGED
|
@@ -3,6 +3,7 @@
|
|
|
3
3
|
[](https://www.npmjs.com/package/fast-uri)
|
|
4
4
|
[](https://github.com/fastify/fast-uri/actions/workflows/ci.yml)
|
|
5
5
|
[](https://github.com/neostandard/neostandard)
|
|
6
|
+
[](https://scorecard.dev/viewer/?uri=github.com/fastify/fast-uri)
|
|
6
7
|
|
|
7
8
|
Dependency-free RFC 3986 URI toolbox.
|
|
8
9
|
|
package/SECURITY.md
ADDED
|
@@ -0,0 +1,173 @@
|
|
|
1
|
+
# Security Policy
|
|
2
|
+
|
|
3
|
+
This document describes the management of vulnerabilities for `fast-uri`, an
|
|
4
|
+
official Fastify project package. It supplements the Fastify project's
|
|
5
|
+
[organization-wide security policy](https://github.com/fastify/.github/blob/main/SECURITY.md).
|
|
6
|
+
|
|
7
|
+
## Standards and scope
|
|
8
|
+
|
|
9
|
+
`fast-uri` implements URI parsing and normalization according to
|
|
10
|
+
[RFC 3986](https://www.rfc-editor.org/rfc/rfc3986). It does not implement the
|
|
11
|
+
[WHATWG URL Standard](https://url.spec.whatwg.org/).
|
|
12
|
+
|
|
13
|
+
Differences between `fast-uri` and WHATWG URL implementations are expected.
|
|
14
|
+
Reports based solely on comparing or mixing parsers that follow these different
|
|
15
|
+
standards are out of scope. Applications must use the same parsing and
|
|
16
|
+
normalization rules for both security decisions and subsequent URI use.
|
|
17
|
+
|
|
18
|
+
Reports are assessed based on reproducibility, documented supported usage, and
|
|
19
|
+
concrete security impact—not merely on whether they mention WHATWG URL
|
|
20
|
+
behavior. Reports demonstrating a parsing or normalization flaw in a supported
|
|
21
|
+
scheme remain eligible for evaluation.
|
|
22
|
+
|
|
23
|
+
## Threat model
|
|
24
|
+
|
|
25
|
+
`fast-uri`'s threat model extends the
|
|
26
|
+
[Node.js threat model](https://github.com/nodejs/node/blob/main/SECURITY.md#the-nodejs-threat-model).
|
|
27
|
+
|
|
28
|
+
**Trusted:** Application code, parser options, configuration, and the runtime
|
|
29
|
+
environment.
|
|
30
|
+
|
|
31
|
+
**Untrusted:** URI and IRI strings passed to the package's public APIs.
|
|
32
|
+
|
|
33
|
+
### Examples of vulnerabilities
|
|
34
|
+
|
|
35
|
+
- RFC 3986 parsing or normalization flaws that bypass security controls
|
|
36
|
+
- Denial of service through malformed input
|
|
37
|
+
- Inconsistent parsing or normalization between `fast-uri` APIs
|
|
38
|
+
|
|
39
|
+
### Examples of non-vulnerabilities
|
|
40
|
+
|
|
41
|
+
The following are **not** considered vulnerabilities in `fast-uri`:
|
|
42
|
+
|
|
43
|
+
- **Different URL standards:** Differences between RFC 3986 behavior and
|
|
44
|
+
WHATWG URL behavior, including differences exposed by mixing parsers that
|
|
45
|
+
implement those standards
|
|
46
|
+
- **Unsupported scheme semantics:** Scheme-specific behavior for schemes that
|
|
47
|
+
`fast-uri` does not document as supported
|
|
48
|
+
- **Application code vulnerabilities:** Security flaws in code that consumes
|
|
49
|
+
`fast-uri` output
|
|
50
|
+
- **Configuration mistakes:** Security issues caused by incorrect parser
|
|
51
|
+
options or application configuration
|
|
52
|
+
- **Missing security features:** Application-level protections that are not
|
|
53
|
+
part of URI parsing or normalization
|
|
54
|
+
- **Third-party dependencies:** Vulnerabilities in packages used by an
|
|
55
|
+
application alongside `fast-uri`
|
|
56
|
+
|
|
57
|
+
## Reporting vulnerabilities
|
|
58
|
+
|
|
59
|
+
Individuals who find potential vulnerabilities in `fast-uri` are invited to
|
|
60
|
+
complete a vulnerability report via the
|
|
61
|
+
[GitHub Security page][advisory].
|
|
62
|
+
|
|
63
|
+
Do not assign or request a CVE directly.
|
|
64
|
+
CVE assignment is handled by the Fastify Security Team.
|
|
65
|
+
Fastify falls under the [OpenJS CNA](https://cna.openjsf.org/).
|
|
66
|
+
A CVE will be assigned as part of our responsible disclosure process.
|
|
67
|
+
|
|
68
|
+
> [!NOTE]
|
|
69
|
+
> Fastify's [HackerOne](https://hackerone.com/fastify) program is now closed.
|
|
70
|
+
|
|
71
|
+
[advisory]: ../../security/advisories/new
|
|
72
|
+
|
|
73
|
+
### Strict measures when reporting vulnerabilities
|
|
74
|
+
|
|
75
|
+
It is of the utmost importance that you read carefully and follow these
|
|
76
|
+
guidelines to ensure the ecosystem as a whole is not disrupted due to
|
|
77
|
+
improperly reported vulnerabilities:
|
|
78
|
+
|
|
79
|
+
- Avoid creating new "informative" reports. Only create a new report if you are
|
|
80
|
+
confident the issue is an actual vulnerability. Third-party vendors and
|
|
81
|
+
individuals track new GitHub security reports and may flag them for their
|
|
82
|
+
customers.
|
|
83
|
+
- Security reports should never be created and triaged by the same person. A
|
|
84
|
+
second Security Team member must triage a report submitted by a team member
|
|
85
|
+
or on their behalf. If in doubt, invite more Fastify collaborators to help
|
|
86
|
+
review the report.
|
|
87
|
+
- **Do not** attempt to demonstrate CI/CD vulnerabilities by creating pull
|
|
88
|
+
requests in Fastify organization repositories. Doing so will result in a
|
|
89
|
+
[content report][content-report] to GitHub as an unsolicited exploit. Create
|
|
90
|
+
a separate repository configured like the affected repository and provide
|
|
91
|
+
the proof of concept there instead.
|
|
92
|
+
|
|
93
|
+
[content-report]: https://docs.github.com/en/communities/maintaining-your-safety-on-github/reporting-abuse-or-spam#reporting-an-issue-or-pull-request
|
|
94
|
+
|
|
95
|
+
### Vulnerabilities found outside this process
|
|
96
|
+
|
|
97
|
+
The Fastify project does not support vulnerability reporting outside the
|
|
98
|
+
process described in this document.
|
|
99
|
+
|
|
100
|
+
## Handling vulnerability reports
|
|
101
|
+
|
|
102
|
+
When a potential vulnerability is reported, the following actions are taken.
|
|
103
|
+
|
|
104
|
+
### Triage
|
|
105
|
+
|
|
106
|
+
**Delay:** 4 business days
|
|
107
|
+
|
|
108
|
+
Within 4 business days, a member of the Security Team provides a first response
|
|
109
|
+
to the reporter. The possible responses are:
|
|
110
|
+
|
|
111
|
+
- **Acceptance:** The report is considered a new vulnerability.
|
|
112
|
+
- **Rejection:** The report is not considered a new vulnerability.
|
|
113
|
+
- **Need more information:** The Security Team needs more information to
|
|
114
|
+
evaluate the report.
|
|
115
|
+
|
|
116
|
+
Triaging should include updating these issue fields:
|
|
117
|
+
|
|
118
|
+
- Asset: set or create the module affected by the report
|
|
119
|
+
- Severity: TBD, initially left empty
|
|
120
|
+
|
|
121
|
+
### Correction follow-up
|
|
122
|
+
|
|
123
|
+
**Delay:** 90 days
|
|
124
|
+
|
|
125
|
+
When a vulnerability is confirmed, a member of the Security Team volunteers to
|
|
126
|
+
follow up on the report.
|
|
127
|
+
|
|
128
|
+
With the reporter's help, they contact the maintainers of the vulnerable
|
|
129
|
+
package and may invite them as participants in the report. Together, they
|
|
130
|
+
define a publication date. Ideally, publication should not happen before the
|
|
131
|
+
package has been patched.
|
|
132
|
+
|
|
133
|
+
The report's vulnerable-version upper limit should be set to:
|
|
134
|
+
|
|
135
|
+
- `*` if no fixed version is available when the report is published
|
|
136
|
+
- the last vulnerable version, such as `<=1.2.3`, if the fix is released in
|
|
137
|
+
`1.2.4`
|
|
138
|
+
|
|
139
|
+
### Publication
|
|
140
|
+
|
|
141
|
+
**Delay:** 90 days
|
|
142
|
+
|
|
143
|
+
Within 90 days after triage, the vulnerability must be made public.
|
|
144
|
+
|
|
145
|
+
Vulnerability severity is assessed using
|
|
146
|
+
[CVSS v3](https://www.first.org/cvss/user-guide).
|
|
147
|
+
|
|
148
|
+
If package maintainers are actively developing a patch, the Security Team and
|
|
149
|
+
the reporter may approve an additional delay.
|
|
150
|
+
|
|
151
|
+
### Secondary contact
|
|
152
|
+
|
|
153
|
+
If you do not receive an acknowledgment within 6 business days, or cannot find
|
|
154
|
+
a private security contact, contact the OpenJS Foundation CNA at
|
|
155
|
+
<https://cna.openjsf.org/> or `security@lists.openjsf.org` for assistance.
|
|
156
|
+
|
|
157
|
+
The CNA can help ensure that reports are acknowledged, coordinate disclosure
|
|
158
|
+
timelines, and assign CVEs when necessary.
|
|
159
|
+
|
|
160
|
+
## The Fastify Security Team
|
|
161
|
+
|
|
162
|
+
The core team manages the security program, policy, and process.
|
|
163
|
+
|
|
164
|
+
Team members must keep all privileged information private. This includes the
|
|
165
|
+
existence of undisclosed issues, expected release dates, and upcoming patches.
|
|
166
|
+
|
|
167
|
+
### Members
|
|
168
|
+
|
|
169
|
+
- [Matteo Collina](https://github.com/mcollina)
|
|
170
|
+
- [Tomas Della Vedova](https://github.com/delvedor)
|
|
171
|
+
- [Vincent Le Goff](https://github.com/zekth)
|
|
172
|
+
- [KaKa Ng](https://github.com/climba03003)
|
|
173
|
+
- [James Sumners](https://github.com/jsumners)
|
package/benchmark/benchmark.mjs
CHANGED
|
@@ -10,6 +10,21 @@ const ipv6 = '//[2001:db8::7]'
|
|
|
10
10
|
const urn = 'urn:foo:a123,456'
|
|
11
11
|
const urnuuid = 'urn:uuid:f81d4fae-7dec-11d0-a765-00a0c91e6bf6'
|
|
12
12
|
|
|
13
|
+
const mailtoSimple = 'mailto:chris@example.com'
|
|
14
|
+
const mailtoWithSubject = 'mailto:infobot@example.com?subject=current-issue'
|
|
15
|
+
const mailtoWithBody = 'mailto:infobot@example.com?body=send%20current-issue%0D%0Asend%20index'
|
|
16
|
+
const mailtoWithHeaders = 'mailto:list@example.org?In-Reply-To=%3C3469A91.D10AF4C@example.com%3E'
|
|
17
|
+
const mailtoMultiple = 'mailto:joe@example.com,alice@example.org?subject=Test&body=NATTO'
|
|
18
|
+
const mailtoEncoded = 'mailto:%22oh%5C%5Cno%22@example.org'
|
|
19
|
+
|
|
20
|
+
const mailtoComponent = {
|
|
21
|
+
scheme: 'mailto',
|
|
22
|
+
to: ['chris@example.com'],
|
|
23
|
+
subject: 'current-issue',
|
|
24
|
+
body: 'send current-issue\r\nsend index',
|
|
25
|
+
headers: { 'In-Reply-To': '<3469A91.D10AF4C@example.com>' }
|
|
26
|
+
}
|
|
27
|
+
|
|
13
28
|
const urnuuidComponent = {
|
|
14
29
|
scheme: 'urn',
|
|
15
30
|
nid: 'uuid',
|
|
@@ -146,6 +161,62 @@ benchUriJs.add('urijs: equal', function () {
|
|
|
146
161
|
uriJsEqual('example://a/b/c/%7Bfoo%7D', 'eXAMPLE://a/./b/../b/%63/%7bfoo%7d')
|
|
147
162
|
})
|
|
148
163
|
|
|
164
|
+
benchFastUri.add('fast-uri: parse mailto simple', function () {
|
|
165
|
+
fastUriParse(mailtoSimple)
|
|
166
|
+
})
|
|
167
|
+
benchUriJs.add('urijs: parse mailto simple', function () {
|
|
168
|
+
uriJsParse(mailtoSimple)
|
|
169
|
+
})
|
|
170
|
+
|
|
171
|
+
benchFastUri.add('fast-uri: parse mailto subject', function () {
|
|
172
|
+
fastUriParse(mailtoWithSubject)
|
|
173
|
+
})
|
|
174
|
+
benchUriJs.add('urijs: parse mailto subject', function () {
|
|
175
|
+
uriJsParse(mailtoWithSubject)
|
|
176
|
+
})
|
|
177
|
+
|
|
178
|
+
benchFastUri.add('fast-uri: parse mailto body CRLF', function () {
|
|
179
|
+
fastUriParse(mailtoWithBody)
|
|
180
|
+
})
|
|
181
|
+
benchUriJs.add('urijs: parse mailto body CRLF', function () {
|
|
182
|
+
uriJsParse(mailtoWithBody)
|
|
183
|
+
})
|
|
184
|
+
|
|
185
|
+
benchFastUri.add('fast-uri: parse mailto headers', function () {
|
|
186
|
+
fastUriParse(mailtoWithHeaders)
|
|
187
|
+
})
|
|
188
|
+
benchUriJs.add('urijs: parse mailto headers', function () {
|
|
189
|
+
uriJsParse(mailtoWithHeaders)
|
|
190
|
+
})
|
|
191
|
+
|
|
192
|
+
benchFastUri.add('fast-uri: parse mailto multi recipient', function () {
|
|
193
|
+
fastUriParse(mailtoMultiple)
|
|
194
|
+
})
|
|
195
|
+
benchUriJs.add('urijs: parse mailto multi recipient', function () {
|
|
196
|
+
uriJsParse(mailtoMultiple)
|
|
197
|
+
})
|
|
198
|
+
|
|
199
|
+
benchFastUri.add('fast-uri: parse mailto encoded local', function () {
|
|
200
|
+
fastUriParse(mailtoEncoded)
|
|
201
|
+
})
|
|
202
|
+
benchUriJs.add('urijs: parse mailto encoded local', function () {
|
|
203
|
+
uriJsParse(mailtoEncoded)
|
|
204
|
+
})
|
|
205
|
+
|
|
206
|
+
benchFastUri.add('fast-uri: serialize mailto', function () {
|
|
207
|
+
fastUriSerialize(mailtoComponent)
|
|
208
|
+
})
|
|
209
|
+
benchUriJs.add('urijs: serialize mailto', function () {
|
|
210
|
+
uriJsSerialize(mailtoComponent)
|
|
211
|
+
})
|
|
212
|
+
|
|
213
|
+
benchFastUri.add('fast-uri: serialize+parse mailto round-trip', function () {
|
|
214
|
+
fastUriSerialize(fastUriParse(mailtoWithBody))
|
|
215
|
+
})
|
|
216
|
+
benchUriJs.add('urijs: serialize+parse mailto round-trip', function () {
|
|
217
|
+
uriJsSerialize(uriJsParse(mailtoWithBody))
|
|
218
|
+
})
|
|
219
|
+
|
|
149
220
|
await benchFastUri.run()
|
|
150
221
|
console.log(benchFastUri.name)
|
|
151
222
|
console.table(benchFastUri.table())
|