Open
Conversation
…attachments, configure node debugger, etc..
…canned and cleanly running thru clamav with no errors..
…ood and bad attachments..
… and code clean-up..
…ully resolve failures in clamav instead of crashing..
…ded, need to keep app afloat, this finishes 1753 ticket..
…s in clamav-ts file..
…s in clamav-ts file..
…an-pix-1754' into attach-scan-pix-1755
…kend still needs touch ups as it hangs..
…, all good so far...
…tuning to follow..
…ng multiple file ok too..
…ut looking good now..
jmckomni
approved these changes
Apr 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Back-End Attachment Scan Work & Toasts POC: 1756
Summary
Implements back-end pre-disk scanning for ALL accepted file types (and rejection handling for unaccepted/infected files or types), and UI messaging/toaster POC for the pre-disk ClamAV attachment scanning epic. This work includes previous tickets
#1751, #1752, #1753, #1754 and #1755— consolidated into this PR, #1756.***Of Note: No
API contractwork is included in this branch, and, this branch does not handle front-end messaging whenspecific file types are checked-offduring Form creation; that specific issue will be addressed in subsequent branches that immediately follow ***`Key Features & Behavior
ClamAV Scan
eicar.txt) are rejected with a clear error message and toast.Front-End
TESTING REQS & SETUP
AWS SSO Login and ClamAV Forwarding
To run the pre-disk ClamAV scan process, ensure the following is in place:
docker-compose.ymljust underSFTP_PLUGIN_CONFIG_SALT:CLAM_AV_URL: tcp://host.docker.internal:3310Make certain you export the new required env variable in a current running session terminal window or VS Code terminal:
export CLAM_AV_URL="tcp://host.docker.internal:3310"AWS SSO Login
magegov.RUN:
aws sso login --profile magegovKube config set up if required for the following
RUN:
kubectl port-forward svc/clamav 3310:3310 -n clamavTesting
EICAR.txtfiles get rejected.Conclusion
This PR covers all back-end work for pre-disk ClamAV attachment scanning, plus POC UI messaging/toaster implementation.
Of Note: NO API Contract work is included in this branch. All of that work track will be in the next series of tickets.