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

[IMP] mail: control file type to upload #1

Merged

Conversation

ndd-odoo
Copy link
Member

@ndd-odoo ndd-odoo commented Jun 25, 2024

-Before this commit: user can upload any file type they want, problem is someone might upload virus file like html one which contain toxic code
-After this commit: provide a way to config which file types is blacklist (can't be uploaded) by using system parameter call "mail.blacklist_file_types"

Description of the issue/feature this PR addresses: https://viindoo.com/web#id=92023&cids=1&menu_id=89&model=project.task&view_type=form

video-2024-06-25_09.28.02.mp4

I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@ndd-odoo ndd-odoo force-pushed the v15_imp_mail_blacklist_file_upload branch from b21e003 to 93209de Compare June 25, 2024 02:22
-Before this commit: user can upload any file type they want, problem is
someone might upload virus file like html one which contain toxic code
-After this commit: provide a way to config which file types is
blacklist (can't be uploaded) by using system parameter call
"mail.blacklist_file_types"
@ndd-odoo ndd-odoo force-pushed the v15_imp_mail_blacklist_file_upload branch from 93209de to cf5afd9 Compare June 25, 2024 02:25
@viinbot viinbot added 🚀 Queue PR/Commit is added in build queue 🚀 Building PR/Commit is start building all step in script ⚠️ Failed PR/Commit is failed when run script and removed 🚀 Queue PR/Commit is added in build queue 🚀 Building PR/Commit is start building all step in script labels Jun 25, 2024
@viinbot
Copy link

viinbot commented Jun 25, 2024

@duong77476-viindoo Viindoo Test Suite has failed!

@viinbot viinbot added 🚀 Queue PR/Commit is added in build queue 🚀 Building PR/Commit is start building all step in script and removed ⚠️ Failed PR/Commit is failed when run script 🚀 Queue PR/Commit is added in build queue labels Jun 25, 2024
@viinbot
Copy link

viinbot commented Jun 25, 2024

@duong77476-viindoo Viindoo Test Suite has passed!

@viinbot viinbot added 🚀 Running PR/Commit is done for everything and ready for manually test and removed 🚀 Building PR/Commit is start building all step in script labels Jun 25, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@royle-vietnam royle-vietnam merged commit 4c50579 into Viindoo:15.0 Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 Running PR/Commit is done for everything and ready for manually test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants