Skip to content

96 bug sometimes send mail does not save in the database correctly#97

Merged
Aravinda-HWK merged 2 commits intomainfrom
96-bug-sometimes-send-mail-does-not-save-in-the-database-correctly
Nov 14, 2025
Merged

96 bug sometimes send mail does not save in the database correctly#97
Aravinda-HWK merged 2 commits intomainfrom
96-bug-sometimes-send-mail-does-not-save-in-the-database-correctly

Conversation

@Aravinda-HWK
Copy link
Copy Markdown
Collaborator

📌 Description

This PR is to resolve that some mails are not saving in the database correctly.


🔍 Changes Made

  • Fix the issue in the buffer reader.

✅ Checklist (Email System)

  • Core IMAP commands tested (LOGIN, CAPABILITY, LIST, SELECT, FETCH, LOGOUT).
  • Authentication is tested.
  • Docker build & run validated.
  • Configuration loading verified for default and custom paths.
  • Persistent storage with Docker volume verified.
  • Error handling and logging verified
  • Documentation updated (README, config samples).

🧪 Testing Instructions

To test the server, use the instructions in the README in the test directory.


📷 Screenshots / Logs (if applicable)


⚠️ Notes for Reviewers

- Added `handleFetchForUIDs` and `handleFetch` methods to manage FETCH requests for messages by UID and sequence number.
- Introduced middleware functions for authentication and mailbox selection checks.
- Created response builders for BODYSTRUCTURE and ENVELOPE according to RFC 3501 specifications.
- Enhanced message parsing to support multipart messages and various body formats.
- Implemented error handling for database interactions and message reconstruction.
- Refactored code for better readability and maintainability.
@Aravinda-HWK Aravinda-HWK self-assigned this Nov 12, 2025
@Aravinda-HWK Aravinda-HWK linked an issue Nov 12, 2025 that may be closed by this pull request
@Aravinda-HWK Aravinda-HWK merged commit b851e29 into main Nov 14, 2025
3 checks passed
@Aravinda-HWK Aravinda-HWK deleted the 96-bug-sometimes-send-mail-does-not-save-in-the-database-correctly branch November 14, 2025 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Sometimes send mail does not save in the database correctly

2 participants