-
Notifications
You must be signed in to change notification settings - Fork 394
feat:encode sfn > functionId & extractedFilename #1872
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
base: main
Are you sure you want to change the base?
Conversation
|
✅ Deploy Preview for solid-start-landing-page ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Could probably use a simple hash such as a xxhash32(filename), but otherwise this is fine. |
|
I have come up with a solution that can use hash algorithm,` HandleServerFunction does not require decoding. I need some time to verify |
I don't think we need to decode anything. We just use the IDs for lookup and verification anyways. |
Now,only Encode In addition, I found that |
PR Checklist
Please check if your PR fulfills the following requirements:
'use server'
will cause an error when there is Chinese in the file path #1859What is the current behavior?
What is the new behavior?
Other information
The encoded characters are guaranteed to be ASCII characters, and the path before
src
is removed to improve security and reduce string sizeCurrently encoded as base64, the size increases by 25%, but I don't know if there is a better way