Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Rediffmail references in header shows plus as a space #277

Closed
avinash403 opened this issue Jan 28, 2019 · 2 comments
Closed

Rediffmail references in header shows plus as a space #277

avinash403 opened this issue Jan 28, 2019 · 2 comments
Assignees
@Sebbo94BY
Labels
needs investigation This will be tested / debugged or checked out.

Comments

@avinash403
Copy link

STEPS TO REPRODUCE:
Send an email in gmail with cc as a rediffmail account. Now reply with the rediffmail account. The + sign in in_reply_to show as a space. See the below screenshots for reference
screen shot 2019-01-28 at 4 21 44 pm
screen shot 2019-01-28 at 4 22 04 pm
@Sebbo94BY
Copy link
Collaborator

Allowed characters are: ::alpha::|::digit::|[!#$%&'*+-/=?^_{}|~.]` (RFC2822)
Reference: https://stackoverflow.com/questions/594330/constraints-on-smtp-message-id

So, there should be no space at all. I'll try to reproduce this.

@avinash403: Do you have any code samples, which you used in order to debug this faster?

@Sebbo94BY Sebbo94BY self-assigned this May 4, 2019
@Sebbo94BY Sebbo94BY added the needs investigation This will be tested / debugged or checked out. label May 4, 2019
Sebbo94BY pushed a commit that referenced this issue May 9, 2019
Issue #277: Added PHPUnit tests to test functionality of decoding MIM…
7b97879 
…E strings
@Sebbo94BY
Copy link
Collaborator

I've added some PHPUnit tests to make sure, that this does not happen. All tests were successful. (See above reference.)

Closing this as there is no bug or issue in php-imap.

Feel free to add comments to this issue, when you still think, that something is not working as expected.

@Sebbo94BY Sebbo94BY mentioned this issue May 9, 2019
Merged
Sebbo94BY added a commit that referenced this issue May 9, 2019
@Sebbo94BY
Some small fixes and improvements
75e203b 
- Issue #313: Fixed skipping of args, when they are identical to their defaults
- Issue #314: Added support for parsing drafts
- Issue #277: Added PHPUnit tests to test functionality of decoding MIME strings
- Issue #253: Added charset property to IncomingMailAttachment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Sebbo94BY Sebbo94BY

Labels
needs investigation This will be tested / debugged or checked out.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Sebbo94BY @avinash403