lixu/goods
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

bypos-collector: 移除硬编码 sdogid、添加重试、补全 CSV、增加测试 #21

Closed
lixu wants to merge 3 commits from devin/1782270248-bypos-collector-improvements into main
pull from: devin/1782270248-bypos-collector-improvements
merge into: lixu:main
Conversation 0 Commits 3 Files Changed 5
+217 -23
lixu commented 2026-06-24 11:11:12 +08:00
Owner
Copy Link
Copy Source

改进内容

安全性

  • sdogid 不再硬编码：优先从环境变量 $BYPOS_SDOGID 读取，其次 -sdogid CLI 参数，也可在 Web 控制台输入框填写。三者都未配置时启动警告、采集时报错拒绝。

可靠性

  • 网络错误自动重试：lookup() 改为最多 3 次重试 + 指数退避（500ms → 1s → 2s），减少因偶发超时丢失条码。

功能补全

  • Web 控制台加 sdogid 输入框：采集参数区新增输入框，无需改命令行即可指定账号。
  • CSV 导出补全字段：export.csv 增加 retmsg（原始返回）和 source（数据源）两列。

测试

  • 新增 collect_test.go：覆盖 ean13Check、md5hex、sanitizeBarcodes、resolveSdogID、空 sdogid 拦截等核心逻辑。

文档

  • README 更新配置说明，反映新的 sdogid 读取优先级。
## 改进内容 ### 安全性 - **sdogid 不再硬编码**：优先从环境变量 `$BYPOS_SDOGID` 读取，其次 `-sdogid` CLI 参数，也可在 Web 控制台输入框填写。三者都未配置时启动警告、采集时报错拒绝。 ### 可靠性 - **网络错误自动重试**：`lookup()` 改为最多 3 次重试 + 指数退避（500ms → 1s → 2s），减少因偶发超时丢失条码。 ### 功能补全 - **Web 控制台加 sdogid 输入框**：采集参数区新增输入框，无需改命令行即可指定账号。 - **CSV 导出补全字段**：`export.csv` 增加 `retmsg`（原始返回）和 `source`（数据源）两列。 ### 测试 - 新增 `collect_test.go`：覆盖 `ean13Check`、`md5hex`、`sanitizeBarcodes`、`resolveSdogID`、空 sdogid 拦截等核心逻辑。 ### 文档 - README 更新配置说明，反映新的 sdogid 读取优先级。
lixu added 1 commit 2026-06-24 11:11:13 +08:00
bypos-collector: 移除硬编码 sdogid、添加重试、补全 CSV 导出、增加测试
CI / Go (api) (pull_request) Failing after 1s
Details
CI / Python (ingestion) (pull_request) Failing after 0s
Details
CI / Migrations (postgres) (pull_request) Failing after 2s
Details
21c895a008 
改进内容:
- sdogid 不再硬编码，优先从 $BYPOS_SDOGID 环境变量读取，其次 -sdogid 参数，
  也可在 Web 控制台输入框填写；三者都未设时启动警告、采集报错
- 网络错误自动重试（最多 3 次，指数退避 500ms/1s/2s）
- Web 控制台采集参数区增加 sdogid 输入框
- CSV 导出补全 retmsg 和 source 两列
- 新增 Go 单元测试（ean13Check、md5hex、sanitizeBarcodes、resolveSdogID 等）
- README 更新配置说明

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
lixu added 1 commit 2026-06-24 11:23:16 +08:00
fix: Windows 闪退 — 增加日志文件输出、错误时保持窗口
CI / Go (api) (pull_request) Failing after 2s
Details
CI / Python (ingestion) (pull_request) Failing after 0s
Details
CI / Migrations (postgres) (pull_request) Failing after 2s
Details
5a641705dd 
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
lixu added 1 commit 2026-06-24 12:18:22 +08:00
fix: 端口占用时自动选择可用端口，不再闪退
CI / Go (api) (pull_request) Failing after 1s
Details
CI / Python (ingestion) (pull_request) Failing after 1s
Details
CI / Migrations (postgres) (pull_request) Failing after 1s
Details
cbb0968256 
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
lixu closed this pull request 2026-06-24 12:44:23 +08:00
Some checks are pending
CI / Go (api) (pull_request) Failing after 1s
Details
CI / Python (ingestion) (pull_request) Failing after 1s
Details
CI / Migrations (postgres) (pull_request) Failing after 1s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
lixu
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: lixu/goods#21
Delete Branch "devin/1782270248-bypos-collector-improvements"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?