Skip to content

Conversation

simonregn
Copy link
Contributor

I don't want to get credits myself. I've copied your changes @lucdew to create this PR, so everybody can benefit from the changes you made. I didn't test it myself

Copy link

@kidager kidager left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, this is breaking smtp at every apply on 26.2

@neistow
Copy link

neistow commented Sep 10, 2025

@kidager Any updates on this PR?

@RafaelWO
Copy link

Note that #1293 adds an option for token auth and would also set smtpServer.AuthType = "basic".

Since this existing auth type can only be basic, I tend towards simply setting it in the provider without giving the user the option to "change" it. AFAIS, this is done in #1293. I.e., if the "OAUTH" PR is accepted, it could potentially supersede this PR? 🤔

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.

5 participants