Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Path is underspecified #1064

Open
1 task done
nichtich opened this issue Dec 13, 2024 · 1 comment
Open
1 task done

Path is underspecified #1064

nichtich opened this issue Dec 13, 2024 · 1 comment
Assignees
Labels
Milestone

Comments

@nichtich
Copy link
Contributor

What Data Package version are you using?

v2

Describe the Issue

The definition of URL or path is not sufficient as "POSIX path" is not formally specified - the link to Wikipedia leads to examples such as hostname:/directorypath/resource but I doubt this is wanted. Furthermore the description of forbidden pathes could be made more clear:

  • is only / disallowed or every path beginning with /?
  • is only .hidden disallowed or also other names starting with a dot and/or what about path/to/.hidden/file?

Do avoid having to read the full POSIX specification, how about a formal definition of path such as

  • any non-empty sequence of bytes except slash and null byte and not starting with a dot is a valid filename
  • a path is a non-empty sequence of filenames, separated with slash

Participation

  • I am willing to submit a pull request for this issue.
@roll roll added the fix label Dec 23, 2024
@roll roll added this to the v2.1 milestone Dec 23, 2024
@roll
Copy link
Member

roll commented Dec 23, 2024

@nichtich
I agree

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants