-
Notifications
You must be signed in to change notification settings - Fork 61
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
add filename length checker to file obfuscator #191
base: main
Are you sure you want to change the base?
Conversation
Hey @TheBearodactyl 👏 Thanks for the contribution! I noticed that you’ve attempted to truncate the overly long encoded filename. However, I believe it might not be ideal for a hider to modify a user’s data — including filenames — without explicit consent. One potential solution could be to add a toggle in the settings, allowing users to enable or disable truncation. Alternatively, a more robust approach might be to store the overly long filename in a separate file. |
sure, lemme add that rq |
Mind if I switch the PR to a draft for now? Just let me know when you think it's good to go! |
sure
…On Sat, Jan 11, 2025, 7:39 PM deltazefiro ***@***.***> wrote:
Mind if I switch the PR to a draft for now? Just let me know when you
think it's good to go!
—
Reply to this email directly, view it on GitHub
<#191 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A3JG4Y4P3MCQ4OXP42UPEGD2KG2VZAVCNFSM6AAAAABUYWZ4POVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKOBVGQ4TONBTGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
No description provided.