[PR #672] [MERGED] fix: imap cannot update message #690

Closed
opened 2026-02-26 21:32:28 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/dreamhunter2333/cloudflare_temp_email/pull/672
Author: @dreamhunter2333
Created: 6/20/2025
Status: Merged
Merged: 6/20/2025
Merged by: @dreamhunter2333

Base: mainHead: feature/dev


📝 Commits (1)

  • 696223c fix: imap cannot update message

📊 Changes

2 files changed (+116 additions, -53 deletions)

View changed files

📝 smtp_proxy_server/imap_server.py (+112 -49)
📝 smtp_proxy_server/requirements.txt (+4 -4)

📄 Description

User description

#668


PR Type

Bug fix, Enhancement


Description

  • Fixed issue with IMAP server not updating message counts correctly.

  • Implemented dynamic message fetching with batching for better performance.

  • Added _update_message_count method to synchronize message counts.

  • Updated dependencies in requirements.txt for compatibility and improvements.


Changes walkthrough 📝

Relevant files
Enhancement
imap_server.py
Refactor IMAP message fetching and count synchronization 

smtp_proxy_server/imap_server.py

  • Added _update_message_count method to synchronize message counts.
  • Refactored fetch method to use a generator for dynamic email fetching.
  • Introduced batching logic for fetching emails in chunks.
  • Improved error handling and logging for HTTP requests.
  • +113/-50
    Dependencies
    requirements.txt
    Update dependencies for compatibility and improvements     

    smtp_proxy_server/requirements.txt

  • Updated pydantic-settings to version 2.9.1.
  • Updated requests to version 2.32.4.
  • Updated Twisted to version 25.5.0.
  • Updated httpx to version 0.28.1.
  • +4/-4     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.

  • 🔄 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/dreamhunter2333/cloudflare_temp_email/pull/672 **Author:** [@dreamhunter2333](https://github.com/dreamhunter2333) **Created:** 6/20/2025 **Status:** ✅ Merged **Merged:** 6/20/2025 **Merged by:** [@dreamhunter2333](https://github.com/dreamhunter2333) **Base:** `main` ← **Head:** `feature/dev` --- ### 📝 Commits (1) - [`696223c`](https://github.com/dreamhunter2333/cloudflare_temp_email/commit/696223c2d17b536e7938e6da329073a118bd206e) fix: imap cannot update message ### 📊 Changes **2 files changed** (+116 additions, -53 deletions) <details> <summary>View changed files</summary> 📝 `smtp_proxy_server/imap_server.py` (+112 -49) 📝 `smtp_proxy_server/requirements.txt` (+4 -4) </details> ### 📄 Description ### **User description** #668 ___ ### **PR Type** Bug fix, Enhancement ___ ### **Description** - Fixed issue with IMAP server not updating message counts correctly. - Implemented dynamic message fetching with batching for better performance. - Added `_update_message_count` method to synchronize message counts. - Updated dependencies in `requirements.txt` for compatibility and improvements. ___ ### **Changes walkthrough** 📝 <table><thead><tr><th></th><th align="left">Relevant files</th></tr></thead><tbody><tr><td><strong>Enhancement</strong></td><td><table> <tr> <td> <details> <summary><strong>imap_server.py</strong><dd><code>Refactor IMAP message fetching and count synchronization</code>&nbsp; </dd></summary> <hr> smtp_proxy_server/imap_server.py <li>Added <code>_update_message_count</code> method to synchronize message counts.<br> <li> Refactored <code>fetch</code> method to use a generator for dynamic email fetching.<br> <li> Introduced batching logic for fetching emails in chunks.<br> <li> Improved error handling and logging for HTTP requests. </details> </td> <td><a href="https://github.com/dreamhunter2333/cloudflare_temp_email/pull/672/files#diff-b8c023c1bf96f6f2045f83138d9e04362526b416db6470823f54243283fabd31">+113/-50</a></td> </tr> </table></td></tr><tr><td><strong>Dependencies</strong></td><td><table> <tr> <td> <details> <summary><strong>requirements.txt</strong><dd><code>Update dependencies for compatibility and improvements</code>&nbsp; &nbsp; &nbsp; </dd></summary> <hr> smtp_proxy_server/requirements.txt <li>Updated <code>pydantic-settings</code> to version 2.9.1.<br> <li> Updated <code>requests</code> to version 2.32.4.<br> <li> Updated <code>Twisted</code> to version 25.5.0.<br> <li> Updated <code>httpx</code> to version 0.28.1. </details> </td> <td><a href="https://github.com/dreamhunter2333/cloudflare_temp_email/pull/672/files#diff-efff5cff7f49b45b10af7620b7a7e2c5aeaf7147bfc9c3412e0c95229bfb0aa2">+4/-4</a>&nbsp; &nbsp; &nbsp; </td> </tr> </table></td></tr></tr></tbody></table> ___ > <details> <summary> Need help?</summary><li>Type <code>/help how to ...</code> in the comments thread for any questions about PR-Agent usage.</li><li>Check out the <a href="https://qodo-merge-docs.qodo.ai/usage-guide/">documentation</a> for more information.</li></details> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
    kerem 2026-02-26 21:32:28 +03:00
    Sign in to join this conversation.
    No milestone
    No project
    No assignees
    1 participant
    Notifications
    Due date
    The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

    No due date set.

    Dependencies

    No dependencies set.

    Reference
    starred/cloudflare_temp_email#690
    No description provided.