vert-x3/vertx-mail-client issues and pull requests

#50 - Issue48 hostnamenotasync

Pull Request - State: closed - Opened by alexlehm over 8 years ago - 1 comment

#49 - clean up some comments

Issue - State: closed - Opened by alexlehm almost 9 years ago

#49 - clean up some comments

Issue - State: closed - Opened by alexlehm almost 9 years ago

#48 - Utils.getMyHostname is not async

Issue - State: closed - Opened by alexlehm almost 9 years ago - 3 comments
Labels: bug

#48 - Utils.getMyHostname is not async

Issue - State: closed - Opened by alexlehm almost 9 years ago - 3 comments
Labels: bug

#47 - fix issue #46: ehloCmd method is not run when Mail Server reply "ESMTP" is not all uppercase characters

Pull Request - State: closed - Opened by alexlehm almost 9 years ago - 2 comments

#47 - fix issue #46: ehloCmd method is not run when Mail Server reply "ESMTP" is not all uppercase characters

Pull Request - State: closed - Opened by alexlehm almost 9 years ago - 2 comments

#46 - ehloCmd method is not run when Mail Server reply "ESMTP" is not all uppercase characters

Issue - State: closed - Opened by leotu almost 9 years ago - 3 comments
Labels: bug, ready

#46 - ehloCmd method is not run when Mail Server reply "ESMTP" is not all uppercase characters

Issue - State: closed - Opened by leotu almost 9 years ago - 3 comments
Labels: bug, ready

#45 - added unit test for address validation exception (pr#44)

Pull Request - State: closed - Opened by alexlehm about 9 years ago - 1 comment

#45 - added unit test for address validation exception (pr#44)

Pull Request - State: closed - Opened by alexlehm about 9 years ago - 1 comment

#44 - added catching exceptions when starting SMTPSendMail and redirecting …

Pull Request - State: closed - Opened by wrobelm about 9 years ago - 2 comments

#44 - added catching exceptions when starting SMTPSendMail and redirecting …

Pull Request - State: closed - Opened by wrobelm about 9 years ago - 2 comments

#43 - fixed #16 remove idle timeout function

Pull Request - State: closed - Opened by alexlehm about 9 years ago - 1 comment

#43 - fixed #16 remove idle timeout function

Pull Request - State: closed - Opened by alexlehm about 9 years ago - 1 comment

#42 - clarify the shared pool thing a bit

Pull Request - State: closed - Opened by alexlehm about 9 years ago

#42 - clarify the shared pool thing a bit

Pull Request - State: closed - Opened by alexlehm about 9 years ago

#41 - change documentation to mention shared/noshared clients

Pull Request - State: closed - Opened by alexlehm about 9 years ago - 1 comment

#41 - change documentation to mention shared/noshared clients

Pull Request - State: closed - Opened by alexlehm about 9 years ago - 1 comment

#40 - Implemented pool sharing for client

Pull Request - State: closed - Opened by purplefox about 9 years ago - 2 comments

#40 - Implemented pool sharing for client

Pull Request - State: closed - Opened by purplefox about 9 years ago - 2 comments

#39 - Revert "pr for issues #16 and #17"

Pull Request - State: closed - Opened by purplefox about 9 years ago

#39 - Revert "pr for issues #16 and #17"

Pull Request - State: closed - Opened by purplefox about 9 years ago

#38 - Client pool sharing code is not thread safe

Issue - State: closed - Opened by purplefox about 9 years ago - 2 comments

#38 - Client pool sharing code is not thread safe

Issue - State: closed - Opened by purplefox about 9 years ago - 2 comments

#37 - pr for issues #16 and #17

Pull Request - State: closed - Opened by alexlehm over 9 years ago - 3 comments

#36 - JSON properties should be camelCase

Issue - State: closed - Opened by purplefox over 9 years ago - 1 comment

#35 - #34 consolidated merge of wip

Pull Request - State: closed - Opened by cescoffier over 9 years ago

#34 - Merge wip branch, this fixes the issues (except 2), improves the error handling on the SMTPConnection to fix a race condition that sometimes failed some tests

Pull Request - State: closed - Opened by alexlehm over 9 years ago - 2 comments

#33 - Replace replaceAll calls if possible

Issue - State: closed - Opened by alexlehm over 9 years ago - 1 comment

#32 - Determine what MailClosedConnectionTest.java is useful for or get rid of it

Issue - State: closed - Opened by alexlehm over 9 years ago - 1 comment
Labels: test

#31 - Todo issues

Pull Request - State: closed - Opened by alexlehm over 9 years ago

#30 - remove TODO in UtilsTest, we keep the \r\n conversion in the unit test

Issue - State: closed - Opened by alexlehm over 9 years ago - 1 comment
Labels: question, test

#29 - MailEncoderTest should check decoded text

Issue - State: closed - Opened by alexlehm over 9 years ago - 1 comment
Labels: question, test

#28 - allow empty address, this can be used as sender (<>)

Issue - State: closed - Opened by alexlehm over 9 years ago - 4 comments
Labels: question

#27 - TODO: it may be a good idea to construct mock objects e.g. with Mockito

Issue - State: closed - Opened by alexlehm over 9 years ago - 1 comment
Labels: question, test

#26 - add asserts to MailClientImplTest2

Issue - State: closed - Opened by alexlehm over 9 years ago - 1 comment
Labels: test

#25 - AuthOperationFactory contains Auto-generated catch block, report proper exceptions instead

Issue - State: closed - Opened by alexlehm over 9 years ago - 1 comment

#24 - netClientOptions conversion to JSON is not implemented

Issue - State: closed - Opened by alexlehm over 9 years ago - 2 comments
Labels: nothing to do/fixed by other issue fix

#23 - removed TODO in SMTPStarter: we keen the separate classes as is

Issue - State: closed - Opened by alexlehm over 9 years ago - 1 comment

#22 - SMTPSendMail use \r\n as line separator for DaTA instead of \n to be more SMTP compliant

Issue - State: closed - Opened by alexlehm over 9 years ago - 1 comment

#21 - SMTPSendMail do not use replaceAll on the complete message

Issue - State: closed - Opened by alexlehm over 9 years ago - 1 comment

#20 - when sending mails to multiple recipients it is possible that some addresses work and some don't

Issue - State: closed - Opened by alexlehm over 9 years ago - 1 comment

#19 - SMTPConnectionPool: rename method to dataEnded, change setting idle to setIdleTimer

Issue - State: closed - Opened by alexlehm over 9 years ago - 1 comment

#18 - Use Handler<AsyncResult> in SMTPConnectionPool

Issue - State: closed - Opened by alexlehm over 9 years ago - 1 comment

#17 - Client sharing code

Issue - State: closed - Opened by purplefox over 9 years ago - 3 comments

#16 - Do we really need idle timeout?

Issue - State: closed - Opened by purplefox over 9 years ago - 5 comments

#15 - Don't expose NetClientOptions

Issue - State: closed - Opened by purplefox over 9 years ago - 1 comment

#14 - changed comments as per notes for pr13

Pull Request - State: closed - Opened by alexlehm over 9 years ago

#13 - Wip

Pull Request - State: closed - Opened by purplefox over 9 years ago - 4 comments

#12 - Revert "Revert "Merge wip branch, this is mostly the refactorings bra…

Pull Request - State: closed - Opened by purplefox over 9 years ago

#11 - Revert "Merge wip branch, this is mostly the refactorings branch from…

Pull Request - State: closed - Opened by purplefox over 9 years ago

#10 - Merge wip branch, this is mostly the refactorings branch from Tim merged with the master changes for Ruby from Julien

Pull Request - State: closed - Opened by alexlehm over 9 years ago - 4 comments

#9 - Refactorings

Pull Request - State: closed - Opened by alexlehm over 9 years ago - 1 comment

#8 - Refactorings

Pull Request - State: closed - Opened by purplefox over 9 years ago - 2 comments

#7 - Refactorings

Pull Request - State: closed - Opened by purplefox over 9 years ago

#6 - Wip

Pull Request - State: closed - Opened by alexlehm over 9 years ago - 1 comment

#5 - fixed cram-md5 auth, was broken due to errors with byte values

Pull Request - State: closed - Opened by alexlehm over 9 years ago

#4 - fixed cram-md5 auth, was broken due to errors with byte values

Pull Request - State: closed - Opened by alexlehm over 9 years ago - 1 comment

#3 - service descriptor json file naming

Issue - State: closed - Opened by ghost over 9 years ago - 1 comment

#2 - new version of the 1st pr for the project

Pull Request - State: closed - Opened by alexlehm over 9 years ago - 1 comment

#1 - pr for first master version of the tool

Pull Request - State: closed - Opened by alexlehm over 9 years ago - 7 comments

GitHub / vert-x3/vertx-mail-client issues and pull requests