Business Rule - Quarantine Attachments#2385
Business Rule - Quarantine Attachments#2385franks883 wants to merge 3 commits intoServiceNowDevProgram:mainfrom
Conversation
|
Thank you for your contribution. However, it doesn’t comply with our contributing guidelines. As a reminder, the general requirements (as outlined in the CONTRIBUTING.md file) are the following: follow the folder+subfolder+snippetfolder guidelines and include a readme.md file explaining what the code snippet does. Review your contribution against the guidelines and make the necessary adjustments. Closing this for now. Once you make additional changes, feel free to re-open this Pull Request or create a new one. |
|
Hi @snamjosh - Thanks for your feedback, I have reviewed your comment but I cant identify where I've gone wrong here - is it my technical solution that doesn't meet the criteria - or is it the structure of the folder? |
|
In your case, you need to look into existing folder structure https://github.com/ServiceNowDevProgram/code-snippets/tree/main/Server-Side%20Components e.g please don't add script include file under business rule folder. there are separate folders maintained for each object. Take a look at below https://github.com/ServiceNowDevProgram/code-snippets/tree/main |
PR Description:
Adds a Business Rule on sys_attachment that quarantines risky uploads. It detects blocked extensions or oversized files, creates a quarantine record, copies the file across, deletes the original, and logs a concise audit message.
Includes README, a small utility Script Include, and the rule script.
Pull Request Checklist
Overview
Code Quality
Repository Structure Compliance
Core ServiceNow APIs/Server-Side Components/Client-Side Components/Modern Development/Integration/Specialized Areas/Documentation
Restrictions