Skip to content
This repository was archived by the owner on Dec 31, 2024. It is now read-only.

Comments

Update ImapClient.cs#119

Open
pocky6 wants to merge 1 commit intosmiley22:masterfrom
pocky6:patch-1
Open

Update ImapClient.cs#119
pocky6 wants to merge 1 commit intosmiley22:masterfrom
pocky6:patch-1

Conversation

@pocky6
Copy link

@pocky6 pocky6 commented Sep 18, 2015

I'm French (so, sorry for my english). I have mailboxes using accents (eg: "Santé").
MailHeaders can be retrieved from this MailBox but moving emails to it fail.
I suggest to encode the name the same way it's done in fetching headers.
It's my first proposal in GitHub so I don't know if you see it, I suggest to modify in:
public void CopyMessages(IEnumerable uids, string destination, string mailbox = null) {
The retrieval:
string response = SendCommandGetResponse(tag + "UID COPY " + set + " " +
Util.UTF7Encode(destination).QuoteString());
Looks like it works for me.

I'm French (so, sorry for my english). I have mailboxes using accents (eg: "Santé"). 
MailHeaders can be retrieved from this MailBox but moving emails to it fail.
I suggest to encode the name the same way it's done in fetching headers.
It's my first proposal in GitHub so I don't know if you see it, I suggest to modify in:
public void CopyMessages(IEnumerable<uint> uids, string destination, string mailbox = null) {
The retrieval:
string response = SendCommandGetResponse(tag + "UID COPY " + set + " " +
	Util.UTF7Encode(destination).QuoteString());
Looks like it works for me.
@jstedfast
Copy link

This looks like a duplicate of PR #113 and issue #112

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants