polaris64

← The Voice of the Proton Community

My feedback

1 result found

  1. Support for custom IMAP headers (e.g. "X-Keywords") for tagging messages

    12 votes
    2 comments  ·  Proton Mail » New feature  ·  Admin →
    How important is this to you?
    An error occurred while saving the comment
    polaris64 commented  · 

    I can confirm that this is indeed the case. I cloned from https://github.com/ProtonMail/proton-bridge and removed the block referenced in my previous comment. I then tried tagging an e-mail and moving it to the INBOX from the archive. Viewing the headers in the ProtonMail web app now shows: -

    ...
    X-Keywords: test-label
    ...

    This change is not a fix of course as I now have duplicate messages: one in the archive without that header and a copy in the INBOX with that header. But it shows that this part of the code is problematic when tagging modifies an existing message's headers and therefore I believe this to be a bug.

    An error occurred while saving the comment
    polaris64 commented  · 

    I think the issue might be here: https://github.com/ProtonMail/proton-bridge/blob/6ff4c8a73853a3794656e530c6f073e02a22568f/internal/imap/mailbox_append.go#L137

    When mbsync moves a message (e.g. from INBOX to Archive) then it issues an APPEND. However as the message already exists in ProtonMail it looks like the above check prevents the modified message source (with the new X-Keywords header) from being stored.

    polaris64 shared this idea  · 

Feedback and Knowledge Base

(thinking…)