langgenius/dify-official-plugins
1
0
Fork 0
mirror of https://github.com/langgenius/dify-official-plugins.git synced 2026-07-22 01:55:27 -04:00
Code Issues 54 Packages Projects Releases Wiki Activity

[PR #928] [MERGED] Solution for Issue #927: Email File Attachment Error #ieeesoc #1627

New Issue
Closed
opened 2026-02-16 10:23:31 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 10:23:31 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-official-plugins/pull/928
Author: @Satyamgupta2365
Created: 5/18/2025
Status: ✅ Merged
Merged: 5/19/2025
Merged by: @crazywoola

Base: main ← Head: main

📝 Commits (3)

📊 Changes

3 files changed (+489 additions, -192 deletions)

View changed files

📝 tools/email/main.py (+139 -1)
📝 tools/email/manifest.yaml (+147 -33)
📝 tools/email/tools/send_mail.py (+203 -158)

📄 Description

📩 Fix: Robust Handling for Email File Attachments (#927)

This PR addresses Issue #927: Email File Attachment Error by introducing a comprehensive and fault-tolerant mechanism for handling file attachments in the email plugin.

✅ Key Improvements:

  • Enhanced File Type Support
    Supports text, image, audio, and binary files using appropriate MIME types.

  • Error Prevention & Handling

    • Validates file existence and size before attachment.
    • Isolates attachment errors using per-file try/catch blocks to prevent total failure.

  • Detailed Logging

    • Warnings for skipped/invalid files.
    • Error logs for failed attachments to aid in debugging.

  • Robust Exception Management
    Gracefully handles both API-specific and general exceptions with clear error messaging.

This update ensures more reliable email delivery even in the presence of problematic files, improving user experience and system stability.

Close https://github.com/langgenius/dify-official-plugins/issues/927

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langgenius/dify-official-plugins/pull/928 **Author:** [@Satyamgupta2365](https://github.com/Satyamgupta2365) **Created:** 5/18/2025 **Status:** ✅ Merged **Merged:** 5/19/2025 **Merged by:** [@crazywoola](https://github.com/crazywoola) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (3) - [`9a53680`](https://github.com/langgenius/dify-official-plugins/commit/9a536803c039c8098c0827973dced25d35e0a9d0) main.py - [`51227cd`](https://github.com/langgenius/dify-official-plugins/commit/51227cdb2d5680ff99c0127fca87dbee9d8f6ccb) send_mail.py - [`11e3e83`](https://github.com/langgenius/dify-official-plugins/commit/11e3e83c01daaba6584e365166a5b3ddc7ed90e4) manifest.yaml ### 📊 Changes **3 files changed** (+489 additions, -192 deletions) <details> <summary>View changed files</summary> 📝 `tools/email/main.py` (+139 -1) 📝 `tools/email/manifest.yaml` (+147 -33) 📝 `tools/email/tools/send_mail.py` (+203 -158) </details> ### 📄 Description ### 📩 Fix: Robust Handling for Email File Attachments (#927) This PR addresses **Issue #927: Email File Attachment Error** by introducing a comprehensive and fault-tolerant mechanism for handling file attachments in the email plugin. #### ✅ Key Improvements: - **Enhanced File Type Support** Supports text, image, audio, and binary files using appropriate MIME types. - **Error Prevention & Handling** - Validates file existence and size before attachment. - Isolates attachment errors using per-file try/catch blocks to prevent total failure. - **Detailed Logging** - Warnings for skipped/invalid files. - Error logs for failed attachments to aid in debugging. - **Robust Exception Management** Gracefully handles both API-specific and general exceptions with clear error messaging. This update ensures more reliable email delivery even in the presence of problematic files, improving user experience and system stability. Close https://github.com/langgenius/dify-official-plugins/issues/927 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 10:23:31 -05:00
yindo closed this issue 2026-02-16 10:23:31 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
bug
bug
dependencies
documentation
enhancement
good first issue
invalid
pull-request
Mirrored from GitHub Pull Request
 question
stale
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify-official-plugins#1627
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?