getty

getty copied to clipboard

Unable to mock getty.Session interface, due to composition with unexported functions in getty.Connection.

1

**What would you like to be added**: To resolve this problem, it is necessary to modify all the functions defined in getty.Connection to be exported with uppercase names. **Why is...

iSuperCoder

**What this PR does**: upgrade as apache/dubbo-getty v1.4.10

AlexStocks

fix: fix issue detect by doubao

3

**What this PR does**: fix issue93 **Which issue(s) this PR fixes**: for https://github.com/AlexStocks/getty/issues/93 **Special notes for your reviewer**: NONE **Does this PR introduce a user-facing change?**: NONE ## Summary by...

Wangzy455

bug report from doubao

2

从提供的项目代码片段和历史变更记录中，可分析出该项目（getty）存在以下潜在问题或Bug，涵盖代码逻辑、依赖管理、工具配置、安全及可维护性等多个维度： ### 一、代码逻辑与功能风险 #### 1. 日志工具调用潜在错误（`util/logger.go`） ```go // Error func Error(args ...any) { log.Error(args...) } ``` - **风险点**： - **日志库依赖模糊**：代码中`log.Error`未明确`log`的导入路径。若依赖的是Go标准库`log`包（无`Error`方法，仅`Print`/`Fatal`等），会直接导致编译失败；即使依赖第三方库（如`zap`/`logrus`，从`go.sum`看存在`zap`），也存在参数类型不兼容风险——例如`zap.Error`需接收`error`类型作为第一个参数，而此处传`...any`可能导致日志打印异常（如参数解析错误、信息缺失）。 - **无错误处理**：未捕获`log.Error`可能返回的错误（部分日志库会返回写入失败错误），若日志写入磁盘/远程时出错，无法感知和处理，导致关键错误日志丢失。 #### 2. Session并发计数潜在竞态（变更记录2016/08/24） - 变更描述：删除`Session.wg`（`atomic.WaitGroup`），改用`Session.grNum`替代以避免`Close()`阻塞。 - **风险点**： -...

AlexStocks

这个框架有交流群么

2

这个框架有交流群么

ayamzh