-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[v16] Move Server Scripts to the file system #26385
Labels
Comments
rmehta
changed the title
Move Server Scripts to the file system
[v16] Move Server Scripts to the file system
May 9, 2024
Slightly modified proposal:
Execution:
Pros:
Cons:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Why
No matter how much we secure, server scripts will be vulnerable to edits since they directly live in the DB. Server Scripts are absolutely necessary, so we can't kill them either. Maybe we need to move them to a different place which is outside Frappe API. Also RestrictedPython is lame
How
More
Should also apply to reports, print templates, email templates
cc: @sagarvora @ankush @akhilnarang
The text was updated successfully, but these errors were encountered: