Skip to content

DE-10765: Client - Fix global test_mode enabled when set to false - #399

Merged
alex-leb merged 1 commit into
masterfrom
DE-10765
Sep 29, 2026
Merged

alex-leb merged 1 commit into
masterfrom
DE-10765

Conversation

@alex-leb

Copy link
Copy Markdown
Contributor

Client#initialize defaulted test_mode to !Mailgun.test_mode.nil?, which is true for any non-nil value, so Mailgun.test_mode = false enabled test mode. Restore the 1.4.1 behaviour: test mode is on only when Mailgun.test_mode is truthy.

Regression from 3c9c497. Fixes #394.

Client#initialize defaulted test_mode to !Mailgun.test_mode.nil?, which
is true for any non-nil value, so Mailgun.test_mode = false enabled test
mode. Restore the 1.4.1 behaviour: test mode is on only when
Mailgun.test_mode is truthy.

Regression from 3c9c497. Fixes #394.
@alex-leb alex-leb self-assigned this Sep 29, 2026
@alex-leb
alex-leb merged commit 09c6815 into master Sep 29, 2026
8 checks passed
@alex-leb
alex-leb deleted the DE-10765 branch September 29, 2026 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Testmode not correctly initialized from global setting

1 participant