66 commits

Author	SHA1	Message	Date
Winni Neessen	c8a8e9772a	Update test recipient email in client tests ... Changed the test email address from go-mail@mytrashmailer.com to couttifaddebro-1473@yopmail.com. This new address is expected to be used for sending test mails.	2024-10-02 16:25:41 +02:00
Winni Neessen	761e205049	Fix client connection test error handling ... Changed variable assignment in the test to fix error handling. This ensures the error is properly caught and reported during the client connection process.	2024-10-02 13:10:10 +02:00
Winni Neessen	547f78dbee	Enhance SMTP LOGIN auth and add comprehensive tests ... Refactored SMTP LOGIN auth to improve compatibility with various server responses, consolidating error handling and response steps. Added extensive tests to verify successful and failed authentication across different server configurations.	2024-10-02 12:37:54 +02:00
Winni Neessen	72b3f53eb7	Add tests for unsupported SCRAM-SHA authentications ... Introduce a new test case `TestClient_AuthSCRAMSHAX_unsupported` to validate handling of unsupported SCRAM-SHA authentication methods. This ensures the client returns the correct errors when setting unsupported auth types.	2024-10-01 20:45:07 +02:00
Winni Neessen	5058fd5222	Add test for SCRAM-SHA authentication failure cases ... Implemented tests for various SCRAM-SHA authentication methods including SCRAM-SHA-1, SCRAM-SHA-1-PLUS, SCRAM-SHA-256, and SCRAM-SHA-256-PLUS with invalid credentials. This ensures that the client correctly handles and reports authentication failures.	2024-10-01 17:01:10 +02:00
Winni Neessen	7499bae3eb	Add unit tests for SCRAM-SHA authentication methods ... Introduce `TestClient_AuthSCRAMSHAX` to verify SCRAM-SHA-1 and SCRAM-SHA-256 authentication. These tests validate the creation, connection, and closing of clients with the respective authentication methods using environment-configured credentials.	2024-10-01 16:45:02 +02:00
Winni Neessen	324be9d032	Refactor SCRAM tests to include SHA-256-PLUS ... Updated `TestClient_AuthSCRAMSHA1PLUS_tlsexporter` and `TestClient_AuthSCRAMSHA1PLUS_tlsunique` to test both SCRAM-SHA-1-PLUS and SCRAM-SHA-256-PLUS authentication types. Implemented table-driven tests to improve readability and maintainability.	2024-10-01 16:43:36 +02:00
Winni Neessen	e8f3c444e6	Add SCRAM-SHA1-PLUS authentication tests ... Introduced two new unit tests for SCRAM-SHA1-PLUS authentication with TLS exporter and TLS unique options. These tests ensure proper client creation, connection, and disconnection processes are functioning as expected in online environments.	2024-10-01 15:28:53 +02:00
Winni Neessen	c1f6ef07d4	Skip test cases when client creation fails ... Updated the client creation check to skip test cases if the client cannot be created, instead of marking them as errors. This ensures tests dependent on a successful client creation do not fail unnecessarily but are instead skipped.	2024-09-27 17:09:00 +02:00
Winni Neessen	6e98d7e47d	Reduce message loop iterations and add XOAUTH2 tests ... Loop iterations in `client_test.go` were reduced from 50 to 20 for efficiency. Added new tests to verify XOAUTH2 authentication support and error handling by simulating SMTP server responses.	2024-09-27 17:00:21 +02:00
Winni Neessen	2d98c40cb6	Add concurrent send tests for Client ... Introduced TestClient_DialSendConcurrent_online and TestClient_DialSendConcurrent_local to validate concurrent sending of messages. These tests ensure that the Client's send functionality works correctly under concurrent conditions, both in an online environment and using a local test server.	2024-09-27 14:04:02 +02:00
Winni Neessen	2234f0c5bc	Remove connection field from Client struct ... This commit removes the 'connection' field from the 'Client' struct and updates the related test logic accordingly. By using 'smtpClient.HasConnection()' to check for connections, code readability and maintainability are improved. All necessary test cases have been adjusted to reflect this change.	2024-09-27 11:43:22 +02:00
Winni Neessen	5503be8451	Remove redundant error print statements ... Removed redundant fmt.Printf error print statements for connection read and write errors. This cleans up the test output and makes error handling more streamlined.	2024-09-23 11:26:54 +02:00
Winni Neessen	9a9e0c936d	Remove redundant error handling in test code ... The check for io.EOF and the associated print statement were unnecessary because the loop breaks on any error. This change simplifies the error handling logic in the `client_test.go` file and avoids redundant code.	2024-09-23 11:09:03 +02:00
Winni Neessen	4ee11e8406	Refactor SMTP server port handling in tests ... Modified tests to dynamically compute server ports from a base value, enhancing flexibility and preventing potential conflicts. Updated `simpleSMTPServer` function to accept a port parameter.	2024-09-20 16:44:15 +02:00
Winni Neessen	d5437f6b7a	Remove redundant comments from sleep statements ... Removed unnecessary comments that were clarifying the purpose of sleep statements in the test cases. This makes the code cleaner and easier to maintain by reducing clutter.	2024-09-20 16:23:31 +02:00
Winni Neessen	157c138142	Fix linter errors ... Replaced `err == io.EOF` with `errors.Is(err, io.EOF)` for better error comparison. Removed redundant `break` statements to streamline case logic and improve code readability.	2024-09-20 15:58:51 +02:00
Winni Neessen	482194b4b3	Add TestClient_SendErrorReset to validate SMTP error handling ... This test checks the client's ability to handle SMTP reset errors when sending an email. It verifies the correct error type, ensures it is recognized as a permanent error, and confirms the correct message ID handling.	2024-09-20 15:52:05 +02:00
Winni Neessen	b8f0462ce3	Add error handling tests for SMTP client ... Implemented multiple tests to cover various error scenarios in the SMTP client, including invalid email addresses and data transmission failures. Introduced `failReset` flag in `simpleSMTPServer` to simulate server reset failures.	2024-09-20 15:49:03 +02:00
Winni Neessen	6af6a28f78	Add test for SendError with no encoding ... Introduced `TestClient_SendErrorNoEncoding` to verify client behavior when sending a message without encoding. Adjusted server connection handling for better error reporting and connection closure, replacing abrupt exits with returns where appropriate.	2024-09-20 15:05:43 +02:00
Winni Neessen	fbebcf96d8	Add simple SMTP test server for unit testing ... Implemented a simple SMTP test server to facilitate unit testing. This server listens on a specified address and port, accepts connections, and processes common SMTP commands like HELO, MAIL FROM, RCPT TO, DATA, and QUIT. Several constants related to the server configuration were also added to the test file.	2024-09-20 14:48:24 +02:00
Winni Neessen	508a2f2a6c	Refactor connection handling in tests ... Replaced `getTestConnection` with `getTestConnectionNoTestPort` for tests that skip port configuration, improving connection setup flexibility. Added environment variable support for setting ports in `getTestClient` to streamline port configuration in tests.	2024-09-19 15:21:17 +02:00
Winni Neessen	e1098091c3	Refactor variable names in SMTP client code ... The commit introduces improved variable names in the SMTP client code that are more expressive and better describe the function they perform. This is combined with corrections to some code comments. It's aimed to increase code readability and maintainability. Updates are made across the client_119.go and client.go files to ensure consistent naming conventions.	2024-02-24 22:06:18 +01:00
Winni Neessen	0cf9efcc69	Update SMTP authentication mechanisms and related tests ... Introduced the SMTPAuthNoAuth mechanism in the auth.go file meant for instances where authentication is not required or supported. Adjusted tests in client_test.go to account for this new mechanism. Minor order modifications were also made to the list of supported SMTP AUTH types and respective tests for better consistency. Closes #176	2024-02-22 15:24:19 +01:00
Winni Neessen	1efac7fb67	Add new test cases for client functionalities and clean client code ... Test cases have been added for numerous client functionalities including WithTLSPortPolicy option for the NewClient method, Client.SetSSLPort method, and the Client.DialWithContext method with the fallback port functionality. Minor code simplification has also been performed in the 'SetSSLPort' function in client.go file.	2024-01-31 11:22:01 +01:00
Winni Neessen	4a90c2de62	Add additional SSL and TLS tests in client_test.go ... Enhanced the unit tests in client_test.go by adding more test cases for SSL Port and TLS Port Policy. These additions contribute to more effective coverage of edge cases and increase the robustness of the test suite.	2024-01-25 16:56:15 +01:00
Winni Neessen	6746af4605	Update client tests for message delivery indication ... This commit adds checks in the testing functions to verify the successful implementation of message delivery status. A new attribute `isDelivered` is now being validated in our tests, reinforcing the message tracking system and enhancing the overall reliability of our application.	2024-01-23 11:21:56 +01:00
Nicola Murino	61b9e59dd2	oauth2: add more test cases	2023-06-04 10:47:19 +02:00
sters	e757327e1d	Adding WithDialContextFunc client option	2023-04-19 23:20:33 +09:00
Winni Neessen	6633591b51	Implement Logger interface ... As stated in https://github.com/wneessen/go-mail/pull/102#issuecomment-1411956040 it would be beneficial if, instead of forcing the Go stdlib logger on the user to provide a simple interface and use that for logging purposes. This PR implements this simple log.Logger interface as well as a standard logger that satisfies this interface. If no custom logger is provided, the Stdlog will be used (which makes use of the Go stdlib again). Accordingly, a `Client.WithLogger` and `Client.SetLogger` have been implemented. Same applies for the smtp counterparts.	2023-02-03 10:19:26 +01:00
Winni Neessen	63d8cef8ca	Refactor DSN handling from client.go to smtp.go ... This PR refactors the the DSN (RFC 1891) SMTP client handling, that was introduced in f4cdc61dd0. While most of the Client options stay the same, the whole workaround logic for the SMTP client has been removed and added as part of the SMTP client instead. This was we got rid of the Client's own `mail()`, `rcpt()`, `dsnRcpt()`, `dsnMail()` methods as well as the copies of the `cmd()` and `validateLine()` methods. The Client is now using the proper `Mail()` and `Rcpt()` methods of the SMTP client instead.	2023-01-18 10:30:06 +01:00
Winni Neessen	3922d3f195	Switched copyright header from me to "The go-mail Authors"	2023-01-15 16:14:19 +01:00
Winni Neessen	e36511e90e	Fix tests to not log authentication data ... We don't want to expose SMTP authentication details in tests, therefore the tests have been adjusted a bit	2023-01-14 13:05:04 +01:00
Winni Neessen	34b432a985	Implement debug logging in SMTP client ... Resolves #101. Since we now have full control over the SMTP client we can also access the message input and output. This PR introduces a new debug logging feature. Via the `Client.WithDebugLog` the user can enable this feature. It will then make use of the new `smtp/Client.SetDebugLog` method. Once the flag is set to true, the SMTP client will start logging incoming and outgoing messages to os.Stderr. Log directions will be output accordingly	2023-01-14 12:47:51 +01:00
Winni Neessen	f782f3f4fc	Merge branch 'main' into feature/97_fork-the-netsmtp-package-from-stdlib-into-go-mail	2023-01-12 01:31:36 +01:00
Winni Neessen	2f053ed8ab	#98 Set up FreeBSD tests via CirrusCI ... Need to debug the test-host for cirrus	2023-01-12 01:01:33 +01:00
Winni Neessen	0d6777ab39	Fork net/smpt into go-mail ... As part of #97 we are going to fork the official `net/smtp` package into go-mail to provide us with more flexibility. This commit fulfills the first big step of importing the package into smtp/. Also go-mail's own LoginAuth has been moved from auth/ into smtp/ to be consistent with the stdlib. There are still a couple of open issues (i. e. license adjustments and making golangci-lint happy) but so far all tests already work, which is a good start.	2023-01-10 00:38:42 +01:00
Winni Neessen	862749f863	Fix SMTP AUTH LOGIN method for servers with uncommon success messages ... This fixes #94 and basically reverts d0f0435. As James points out correctly in #94, we should not assume specific responses from the server. As long as the spec is followed and the server returns the correct SMTP code, we should not do our own magic. I've also extended the `getTestConnection` method in client_test.go, so that we can specify more test environment options like `TEST_PORT` and `TEST_TLS_SKIP_VERIFY`. This was needed for testing with the ProtonMail Bridge which listens on a different port and has non-trusted certificates.	2023-01-07 11:31:46 +01:00
Winni Neessen	b12b0bb363	Update client_test.go ... Co-authored-by: iwittkau <iwittkau@users.noreply.github.com>	2023-01-02 22:22:31 +01:00
Winni Neessen	2c7ea3e532	More changes in regard to #90 ... As proposed by @iwittkau the `SendError` type now has a `IsTemp()` method as well indicating to the user if the delivery error is retryable or not. Since we want to use it in the error response from the Client functions like `Send` or `DialAndSend` we need to return the SendError type not only as part of the `*Msg` but also as return value for these methods. Hence, the changes made for #85 been overhauled to return the new error type instead. In the pre Go1.20 version of the `Send()` method we need to return an accumulated version of the SendError type, since we don't have `errors.Join()` and therefore, if more than one error occurred during the delivery we return an ambiguous error reason since we can't tell which of the captured errors is main error. For more details the user can always check the `*Msg.SendError`	2023-01-02 12:14:14 +01:00
Winni Neessen	78df991399	Proposal change for #90 ... Did a complete overhaul of the senderror.go. - the list of `errors.New()` has been replaced with constant itoa error reasons as `SendErrReason`. Instead, the `Error()` method now reports the corresponding error message based on the reason. - The `SendError` received a `Is()` method so that we can use `errors.Is()` for very specific error checking. I.e. we can check for `&SendErrors{Reason: ErrSMTPMailFrom, isTemp: true}`. This provides much more flexibility in the error checking capabilities - A `isTemp` field has been added to the `SendError` type, indicating whether the received error is temporary and can be retried or not. Accordingly, the `*Msg` now has a `SendErrorIsTemp()` method indicating the same. The decision is based on the first 3 characters returned from the SMTP server. If the error code is within the 4xx range, the error is seen as temporary - A test for the SendError type has been added	2023-01-01 14:20:13 +01:00
Winni Neessen	47bff15de9	Introduction of a Msg error type as proposal for #90 ... This PR introduces the `SendError` type which implements the error interface. A new `senderror` field has been added to the `Msg` as well, so introduce this type to it. I've also added different error variables that indicate the different things that can go wrong during mail delivery. These variables can be checked for, for each `Msg` using the `errors.As` method The `Error()` method of `SendError` will return a detailed error string on why the `Msg` could not be delivered. Additionally, `HasSendError()` and `SendError()` methods have been added to `Msg`. While `HasSendError()` simply returns a bool in case a `Msg` failed during delivery, the `SendError()` will return the full `SendError` error interface.	2022-12-31 12:40:42 +01:00
James Elliott	567276d75d	feat: without noop option ... This allows disabling the Noop command during the dial. This is useful for servers which delay potentially unwanted clients when they perform commands other than AUTH.	2022-12-27 00:06:20 +11:00
Winni Neessen	48b4dc6b6c	Fix #85: Client.Send() failing for all messages if one is broken ... `Client.Send()` provides the possibility to send multiple `*Msg` in one go. If one of the `*Msg` caused an error with the sending mail server, we were returning completely, while not processing any `*Msg` that came after the failing message. This PR fixes this behaviour by processing each message first and then return a accumulated error in case any of the `*Msg` processing failed Additionally, this PR separates the `Client.Send()` method into two different versions. One that makes use of the new `errors.Join()` functionality that is introduced with Go 1.20 and one that handles it the old way for any supported version lower than Go 1.20	2022-12-10 13:41:00 +01:00
Winni Neessen	d200e982bf	Make golangci-lint errorlint happy	2022-10-17 18:16:00 +02:00
Winni Neessen	94ad8af58a	Make golangci-lint gofumpt happy	2022-10-17 18:13:00 +02:00
Winni Neessen	f53579fcf9	Closes #54 ... - Adds `DialAndSendWithContext()` and makes `DialAndSend()` use the new method	2022-09-26 10:40:57 +02:00
Winni Neessen	7b030473a6	Final set of unit tests for DSN	2022-09-12 09:31:42 +02:00
Winni Neessen	c52a437f7f	More tests	2022-09-11 22:05:43 +02:00
Winni Neessen	aed71397c0	Implemented DSNs as described in RFC 1891 ... - Added test coverage for the WithDSN* methods - Updated README.md accordingly	2022-09-11 21:07:15 +02:00