feat(goofish): auto-generate cna if cookie is missing to fix search API error#17
Merged
11273 merged 1 commit into11273:mainfrom Jan 22, 2026
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📋 Pull Request 信息 | PR Information
🔗 关联 Issue | Related Issue
📝 变更类型 | Type of Change
请选择适用的变更类型 | Please check the type of change your PR introduces:
📖 变更描述 | Description
Previously, initializing a new Goofish instance without a cookie would result in missing the required 'cna' parameter.
This would cause the search API to return the error:
RGV587_ERROR::SM::哎哟喂,被挤爆啦,请稍后重试
Now, if no cookie is provided, a default 'cna' parameter is automatically generated
during initialization, ensuring the search API works correctly.
什么变更?| What changed?
为什么变更?| Why was this change needed?
如何变更?| How was this implemented?
🧪 测试 | Testing
测试步骤 | Test Steps
测试环境 | Test Environment
测试结果 | Test Results
📸 截图/视频 | Screenshots/Videos
📚 文档 | Documentation
破坏性变更详情 | Breaking Change Details
影响的 API | Affected APIs:
迁移指南 | Migration Guide:
向后兼容性 | Backward Compatibility:
📋 检查清单 | Checklist
代码质量 | Code Quality
测试覆盖 | Test Coverage
文档和示例 | Documentation and Examples
安全和性能 | Security and Performance
🔍 Review 注意事项 | Review Notes
重点关注区域 | Areas of Focus
已知问题 | Known Issues
📦 发布影响 | Release Impact
🏷️ 额外标签 | Additional Labels
建议的标签 | Suggested labels:
bug,enhancement,documentation,dependencies,security,performance,breaking-change✅ 准备就绪确认 | Ready for Review Checklist
PR 提交者签名 | Submitter Signature: @Bugegoo