Skip to content

Conversation

@chrisminett
Copy link
Member

Revert the token authentication changes added in #19 , keeping the deprecation for getConfig().

Although this is essentially a BC break as it removes code, no implementation can have used token auth as it is not live in the Maxemail app.

@chrisminett chrisminett self-assigned this Jul 20, 2025
@chrisminett chrisminett added the patch-version Release CAN be patch version increment label Jul 20, 2025
@codecov
Copy link

codecov bot commented Jul 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.45%. Comparing base (af4fafb) to head (6bcb2bd).

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #21      +/-   ##
============================================
- Coverage     91.59%   91.45%   -0.15%     
+ Complexity       63       61       -2     
============================================
  Files             5        5              
  Lines           238      234       -4     
============================================
- Hits            218      214       -4     
  Misses           20       20              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@chrisminett
Copy link
Member Author

Checking that the token auth code has been correctly reverted:

@chrisminett chrisminett merged commit cd55f38 into main Jul 20, 2025
10 checks passed
@chrisminett chrisminett deleted the revert-token-auth branch July 20, 2025 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

patch-version Release CAN be patch version increment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants