astropy / astropy/pyvo

Refactor the code that produces the name of the files returned by services

Open
#573 1 comment 0 reactions 0 assignees View on GitHub
bug
Dominant language
Python
Stars
88
Forks
64
Avg merge
38m
Merged PRs (30d)
1

Description

Refactor `suggest_dataset_basename` (which has fairly parallel implementations
in ssap and sia on top) to produce correct file names across platforms. Discuss whether for security reasons we'd want to limit the charset to ASCII printables. See related #557

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate the parallel suggest_dataset_basename implementations in the ssap and sia modules, then read related issue #557 for context. Check how the generated names behave across platforms and resolve the open question about limiting characters to ASCII printables; done means both service paths produce correct, safe filenames.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api
Issue type
Refactor
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.