Commit04824e7

committed

fix: SMTP query failure due to SNI mismatch

1 parent1053d19 commit04824e7Copy full SHA for 04824e7

File tree

-1

lines changed

server/api-service/lowcoder-plugins/smtpPlugin/src/main/java/org/lowcoder/plugins
- SmtpPlugin.java

-1

lines changed

Lines changed: 9 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -85,8 +85,10 @@ public Mono<Session> createConnection(SmtpDatasourceConfig connectionConfig) {`
`85`	`85`	`prop.put("mail.smtp.host",connectionConfig.getHost());`
`86`	`86`	`prop.put("mail.smtp.port",connectionConfig.getPort() <=0 ?DEFAULT_PORT :connectionConfig.getPort());`
`87`	`87`	`prop.put("mail.smtp.auth",true);`
	`88`	`+prop.put("mail.smtp.username",connectionConfig.getUsername());`
`88`	`89`	`prop.put("mail.smtp.starttls.enable","true");`
`89`	`90`	`prop.put("mail.smtp.ssl.protocols","TLSv1.2");`
	`91`	`+prop.put("mail.smtp.ssl.checkserveridentity",false);`
`90`	`92`
`91`	`93`	`returnMono.fromSupplier(() ->`
`92`	`94`	`// blocked call`
`@@ -119,9 +121,15 @@ public Mono<DatasourceTestResult> testConnection(SmtpDatasourceConfig connection`
`119`	`121`	`returnsessionMono`
`120`	`122`	`.map(session -> {`
`121`	`123`	`try {`
`122`		`-session.getTransport().connect();`
	`124`	`+if (connectionConfig.getUsername() !=null) {`
	`125`	`+session.getTransport().connect(connectionConfig.getUsername(),connectionConfig.getPassword());`
	`126`	`+ }`
	`127`	`+else {`
	`128`	`+session.getTransport().connect();`
	`129`	`+ }`
`123`	`130`	`returnDatasourceTestResult.testSuccess();`
`124`	`131`	`}catch (MessagingExceptione) {`
	`132`	`+log.debug("SmtpPlugin.testConnection() failed!",e);`
`125`	`133`	`returnDatasourceTestResult.testFail(e);`
`126`	`134`	`}`
`127`	`135`	`})`

Comments

(0)