servo / servo/rust-url

Feature request: add parser boolean option to leave relative paths in the URL.

Open
#899 0 comments 2 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Rust
Stars
1.6k
Forks
406
PR merge metrics
No merged PRs in 30d

Description

Hi all, for a project of mine I'd need this library to optionally leave relative paths unparsed (another reference). This seems unavoidable due to this piece of code in the parsing logic. It would be nice if a parsing boolean option were to be added so that that condition can be bypassed, resulting in ".." be left alone and handled by the server.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in url/src/parser.rs around the parsing condition linked in the issue, and trace how relative paths are handled. Determine how a boolean parsing option would bypass that condition, then verify that relative paths containing ".." remain unparsed and are handled by the server.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
networking
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.