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.

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

sasl_getmesssage: make sure we have a long enough string to pass #2150

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
1 participant
@bagder
Copy link
Member

commented Dec 4, 2017

For pop3/imap/smtp, added test 891 to somewhat verify the pop3
case.

For this, I enhanced the pingpong test server to be able to send back
responses with LF-only instead of always using CRLF.

@bagder

This comment has been minimized.

Copy link
Member Author

commented Dec 4, 2017

Test failures. Need to investigate...

sasl_getmesssage: make sure we have a long enough string to pass
For pop3/imap/smtp, added test 891 to somewhat verify the pop3
case.

For this, I enhanced the pingpong test server to be able to send back
responses with LF-only instead of always using CRLF.

@bagder bagder force-pushed the bagder/pop3_get_message branch from d0e0c54 to 7eb6e08 Dec 5, 2017

@bagder

This comment has been minimized.

Copy link
Member Author

commented Dec 5, 2017

mostly dead apple machines (I cancelled their jobs now) and what looks like a false positive test 1208 error

@bagder bagder closed this in 615edc1 Dec 5, 2017

@bagder bagder deleted the bagder/pop3_get_message branch Dec 5, 2017

JohnDeHelian pushed a commit to JohnDeHelian/curl that referenced this pull request Dec 7, 2017

sasl_getmesssage: make sure we have a long enough string to pass
For pop3/imap/smtp, added test 891 to somewhat verify the pop3
case.

For this, I enhanced the pingpong test server to be able to send back
responses with LF-only instead of always using CRLF.

Closes curl#2150

@lock lock bot locked as resolved and limited conversation to collaborators Jan 18, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.