not log error when ssl disabled

[yihaoDeng]

Description

Issue(s)

• Close/close/Fix/fix/Resolve/resolve: Issue Link

Checklist

Please check the items in the checklist if applicable.

• Is the user manual updated?
• Are the test cases passed and automated?
• Is there no significant decrease in test coverage?

[gemini-code-assist]

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

[Copilot]

Pull request overview

This PR prevents error logging when SSL is disabled by adding an early return condition in the connection callback. The change ensures that SSL/SASL authentication code is only executed when both features are enabled, avoiding spurious errors when either feature is disabled.

Key Changes

• Added early return in uvOnConnectionCb to skip SSL/SASL authentication handling when either feature is disabled
• Reorganized struct fields in SRpcInfo for better grouping of SSL/SASL-related fields

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
source/libs/transport/src/transSvr.c	Added pInst variable and early return condition to prevent SSL/SASL authentication code execution when features are disabled
source/libs/transport/inc/transportInt.h	Reorganized enableSasl field placement and cleaned up comment formatting in SRpcInfo struct

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Missing NULL check for pConn->saslConn before calling saslAuthIsInited. While the early return at line 1662 should prevent this case, defensive programming suggests adding an explicit NULL check similar to line 1027 which uses the pattern: if (conn->saslConn && sslIsInited(conn->pTls) && !saslAuthIsInited(conn->saslConn))

Suggested change

	if (sslIsInited(pConn->pTls) && !saslAuthIsInited(pConn->saslConn)) {
	if (pConn->saslConn && sslIsInited(pConn->pTls) && !saslAuthIsInited(pConn->saslConn)) {