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

🚸 info --redact should replace path parts with placeholders #35

Open
jeftadlvw opened this issue Jun 6, 2024 · 0 comments
Open

🚸 info --redact should replace path parts with placeholders #35

jeftadlvw opened this issue Jun 6, 2024 · 0 comments

Comments

@jeftadlvw
Copy link
Owner

Currently, running info with the --redact flag calculates relative paths from the cwd. This is fine. By default, the repository root and project root should not be different. But if they do, the relative paths still leak the single path names. They should be replaced with placeholders (e.g. letters of the standard English alphabet).

It's also important to consider filesystem case-sensitivity when comparing and mapping path parts to placeholders.

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

No branches or pull requests

1 participant