dbf256 / teamcity-password-reset-plugin Goto Github PK

Instead of placing your settings in config/main-config.xml. You should consider making your own config file like config/teamcity-password-reset-plugin.xml and reading your settings from there. Instead of doing it manually you could also do it like these guys https://github.com/guardian/teamcity-s3-plugin. And have your own settings page and have it write to that file.

I've added your plugin to TeamCity, however for some reason it is expecting issues with sending email on password reset. Please note that email notifications are properly configured and work in other cases (when pressing "Test Connection" button or for successful/failing builds).

TeamCity version: 9.1 (build 36973)
Logs from teamcity-server.log:

[2016-01-06 16:41:47,185]   INFO -   jetbrains.buildServer.SERVER - 1 user(s) were found for [email protected] 
[2016-01-06 16:41:47,191]  ERROR -   jetbrains.buildServer.SERVER - Failed to send message, SMTP config is not valid: Port is not set, Host is not set 

TeamCity does not have good API to send email to a user, but there is a jetbrains.buildServer.notification.MessageSender service which can be used for this task. At the moment this extension is implemented by email notifier only.

You can use jetbrains.buildServer.ExtensionProvider to obtain all registered implementations of MessageSender interface. You can distinguish email sender by looking for "Email Notifier" in return value of describe() method. Not very robust, but still better than implementing own email sending.

And by the way, congratulations for winning the plugin contest!