huggingface / huggingface/dataset-viewer

Unescaped config names with special characters in the URL

Open
#2,992 3 comments 0 reactions 0 assignees View on GitHub
bug name issue P1
Dominant language
Python
Stars
899
Forks
130
Avg merge
9h 26m
Merged PRs (30d)
14

Description

When playing with mlcroissant, we observed the following issue:

[bigcode/commitpackft](https://huggingface.co/datasets/bigcode/commitpackft) has both the configs `c` and `c#`. When going to https://huggingface.co/api/datasets/bigcode/commitpackft/parquet/c#/train/0.parquet, it lists https://huggingface.co/api/datasets/bigcode/commitpackft/parquet/c/train/0.parquet (instead of https://huggingface.co/api/datasets/bigcode/commitpackft/parquet/c%23/train/0.parquet).

Should dataset names / config names be escaped in the URLs?

cc @severo @lhoestq

Contributor guide

Open the contributing guide

Research direction

Reproduce the issue with the dataset-viewer API URL for the c# configuration, then trace the endpoint that constructs the parquet URL. Verify that dataset and configuration names with special characters are URL-escaped, and confirm the response uses c%23 rather than c in the generated link.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api, backend, data
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.