Skip to content

refactor(rpc): improve gRPC client TLS config log formatting#852

Open
maomao94 wants to merge 2 commits intonacos-group:masterfrom
maomao94:master
Open

refactor(rpc): improve gRPC client TLS config log formatting#852
maomao94 wants to merge 2 commits intonacos-group:masterfrom
maomao94:master

Conversation

@maomao94
Copy link

  • Change TLS config check logs from string concatenation to placeholder formatting
  • Fix log output format during RPC client initialization
  • Adjust import location of credentials package to follow code conventions

- Change TLS config check logs from string concatenation to placeholder formatting
- Fix log output format during RPC client initialization
- Adjust import location of credentials package to follow code conventions
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@lingma-agents
Copy link

lingma-agents bot commented Oct 20, 2025

重构 gRPC 客户端 TLS 配置日志及初始化逻辑

变更概述
  • 重构

    • 调整了 google.golang.org/grpc/credentials 包的导入位置,使其符合代码规范。
    • 将 TLS 配置检查日志从字符串拼接方式改为占位符格式化输出,提升日志可读性。
    • 在 RPC 客户端初始化过程中增加日志记录,明确标识客户端创建过程。
  • 问题修复

    • 修复了在 gRPC 客户端创建连接时缺少 TLS 配置信息打印的问题,便于调试和监控 TLS 状态。
变更文件
文件路径 变更说明
common/​remote/​rpc/​grpc_​client.​go 调整了 credentials 包的导入顺序,并改进了 TLS 配置的日志输出格式。
common/​remote/​rpc/​rpc_​client.​go 在 RPC 客户端初始化阶段增加了 info 级别日志,用于标识特定名称的客户端正在被创建。

💡 小贴士

与 lingma-agents 交流的方式

📜 直接回复评论
直接回复本条评论,lingma-agents 将自动处理您的请求。例如:

  • 在当前代码中添加详细的注释说明。

  • 请详细介绍一下你说的 LRU 改造方案,并使用伪代码加以说明。

📜 在代码行处标记
在文件的特定位置创建评论并 @lingma-agents。例如:

  • @lingma-agents 分析这个方法的性能瓶颈并提供优化建议。

  • @lingma-agents 对这个方法生成优化代码。

📜 在讨论中提问
在任何讨论中 @lingma-agents 来获取帮助。例如:

  • @lingma-agents 请总结上述讨论并提出解决方案。

  • @lingma-agents 请根据讨论内容生成优化代码。

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.

2 participants