diff options
| author | Omar Polo <op@omarpolo.com> | 2024-06-10 11:42:22 +0000 |
|---|---|---|
| committer | Omar Polo <op@omarpolo.com> | 2024-06-10 11:42:22 +0000 |
| commit | 7f6c46ae1ea4288c371a85080ff67fc406980241 (patch) | |
| tree | 84c3001c4906791370ec3c7a74e86a5f7a896eb3 /regress/iri_test.c | |
| parent | 3d06af043cdbb58d9ef31ad8f9e5a8f46660e79f (diff) | |
iri: don't error on a '..' component at the start of the path
I choose to out of paranoia, but the algorithm defined in RFC3986
allows for them. So, we should rather remove the leading '..'
component and continue to handle the rest of the path.
Fixes https://github.com/omar-polo/gmid/issues/12
Diffstat (limited to 'regress/iri_test.c')
| -rw-r--r-- | regress/iri_test.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/regress/iri_test.c b/regress/iri_test.c index fa6d578..a0e7466 100644 --- a/regress/iri_test.c +++ b/regress/iri_test.c @@ -201,13 +201,13 @@ main(void) IRI("gemini", "omarpolo.com", "", "", "", ""), "parse paths with a trailing .."); TEST("gemini://omarpolo.com/foo/../..", - FAIL, - empty, - "reject paths that would escape the root"); + PASS, + IRI("gemini", "omarpolo.com", "", "", "", ""), + "parse paths that would escape the root"); TEST("gemini://omarpolo.com/foo/../../", - FAIL, - empty, - "reject paths that would escape the root") + PASS, + IRI("gemini", "omarpolo.com", "", "", "", ""), + "parse paths that would escape the root") TEST("gemini://omarpolo.com/foo/../foo/../././/bar/baz/.././.././/", PASS, IRI("gemini", "omarpolo.com", "", "", "", ""), @@ -271,8 +271,8 @@ main(void) IRI("foo", "bar.com", "", "caffè+macchiato.gmi", "", ""), "can decode"); TEST("foo://bar.com/foo%2F..%2F..", - FAIL, - empty, + PASS, + IRI("foo", "bar.com", "", "", "", ""), "conversion and checking are done in the correct order"); TEST("foo://bar.com/foo%00?baz", FAIL, |