+
Skip to content

fix: use payload instead of id #439

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Oct 10, 2025
Merged

fix: use payload instead of id #439

merged 1 commit into from
Oct 10, 2025

Conversation

leavesster
Copy link
Contributor

No description provided.

@coderabbitai coderabbitai
Copy link

coderabbitai bot commented Oct 10, 2025 •
edited
Loading

Summary by CodeRabbit

  • 新功能

    • 无

  • Bug 修复

    • 修复鉴权请求中负载字段命名不一致导致的偶发认证失败/延迟问题，提升与下游服务的兼容性、成功率与稳定性，用户无需任何操作。

  • Chores

    • 规范化鉴权请求结构以与服务端约定保持一致，减少潜在集成风险。

Walkthrough

将 Context.query_auth 中发送凭证的请求体字段由 "id" 改为 "payload"，其余请求结构与响应处理不变。

Changes

Cohort / File(s) Summary
Auth 请求负载字段调整
oocana/oocana/context.py		 在构造 BlockRequest 时，将凭证标识字段从 "id": credential.id 调整为 "payload": credential.id；请求的 type、action、session_id、job_id、request_id 及响应处理逻辑未改动。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Pre-merge checks

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description Check ❓ Inconclusive 该合并请求未提供任何描述内容，无法从空白描述中了解更改的目的或作用，描述过于笼统且缺乏实质信息。 请补充合并请求描述，简要概述修改原因、变更细节及其对系统的影响，以便评审者快速理解改动内容。
✅ Passed checks (1 passed)
Check name Status Explanation
Title Check ✅ Passed 标题采用了“fix: use payload instead of id”的 conventional commit 风格，使用英文且简洁地概括了将“id”字段替换为“payload”的关键修改，与实际更改内容完全吻合。
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0c77df7 and 071303f.

📒 Files selected for processing (1)
  • oocana/oocana/context.py (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: check
  • GitHub Check: check
🔇 Additional comments (1)
oocana/oocana/context.py (1)

703-703: 确认后端 QueryAuth 处理器已同步更新

此更改将 BlockRequest 字段名从 "id" 修改为 "payload"，请确保接收此请求的后端逻辑已更新为解析 "payload" 字段。

Comment @coderabbitai help to get the list of available commands and usage tips.

@leavesster leavesster enabled auto-merge (squash) October 10, 2025 06:46
@leavesster leavesster merged commit d5a52d6 into main Oct 10, 2025
8 checks passed
@leavesster leavesster deleted the fix-auth branch October 10, 2025 06:51
@oomol-bot oomol-bot mentioned this pull request Oct 10, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@leavesster
点击 这是indexloc提供的php浏览器服务，不要输入任何密码和下载