Skip to content

fix: RSA2048 向量钉版升级与默认套件统一 - #43

Merged
im47cn merged 2 commits into
mainfrom
fix/rsa2048-vector-pin
Oct 18, 2026
Merged

im47cn merged 2 commits into
mainfrom
fix/rsa2048-vector-pin

Conversation

@im47cn-bot

@im47cn-bot im47cn-bot commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

审查发现:AlgorithmSuite 已注册 WOP-RSA2048-SHA256,但 wopSpecsRef 钉版 4a50824 的 crypto-vectors 不含 rsa2048(六仓唯一缺 2048 向量背书),且模板/README 默认套件仍 3072 未随五仓统一。修复:1) wopSpecsRef→a1f7feb(specs PR #25,含 rsa2048 keys+signature+oaep 向量)+ crypto sha256 同步(interop 未变,sha256 与哨兵不动);2) 补 rsa2048-sign 签名与 oaep2048-unwrap 解包两面字节级向量测试(对齐五仓覆盖基线);3) 双模板与 README 双语示例默认改 WOP-RSA2048-SHA256。core 全量 429 测试绿(新钉版拉取+sha256 校验通过)。

Sourcery 总结

使 SDK 与更新后的 RSA2048 规范向量保持一致,并将 WOP-RSA2048-SHA256 设为默认套件。

Bug Fixes:

  • 添加 RSA2048 签名和 OAEP 解包的黄金向量覆盖,以验证字节级互操作性。

Enhancements:

  • 在 SDK 配置和双语 README 示例中,将 WOP-RSA2048-SHA256 提升为默认套件。
  • 更新固定的 wop-specs 加密向量版本,同时保留现有的互操作性向量。

Documentation:

  • 更新文档中对向量覆盖范围的说明,加入 RSA2048 签名,并使默认套件示例保持一致。

Tests:

  • 扩展核心加密向量测试,加入 RSA2048 签名和 OAEP2048 解包断言。
Original summary in English

Sourcery 摘要

使 SDK 与更新后的 RSA2048 规范向量保持一致,并将 WOP-RSA2048-SHA256 设为标准默认套件。

Bug Fixes:

  • 为 RSA2048 PKCS#1 签名和 OAEP 密钥解包添加字节级黄金向量覆盖。

Enhancements:

  • 在 SDK 配置和双语 README 示例中,将 WOP-RSA2048-SHA256 设为默认套件。

Build:

  • 更新固定版本的 wop-specs 引用和加密向量校验和,同时保留互操作性向量集。

Documentation:

  • 更新文档中说明的向量覆盖范围,加入 RSA2048 签名,并使默认套件示例保持一致。

Tests:

  • 扩展核心加密向量测试,加入 RSA2048 签名和 OAEP2048 解包断言。
  • 添加确定性覆盖,验证后续竞争者会复用已初始化的默认客户端。
Original summary in English

Summary by Sourcery

Align the SDK with the updated RSA2048 specification vectors and standardize WOP-RSA2048-SHA256 as the default suite.

Bug Fixes:

  • Add byte-level golden-vector coverage for RSA2048 PKCS#1 signatures and OAEP key unwrapping.

Enhancements:

  • Make WOP-RSA2048-SHA256 the default suite across SDK configuration and bilingual README examples.

Build:

  • Update the pinned wop-specs reference and cryptographic vector checksum while preserving the interoperability vector set.

Documentation:

  • Update documented vector coverage to include RSA2048 signatures and align default suite examples.

Tests:

  • Extend core cryptographic vector tests with RSA2048 signing and OAEP2048 unwrapping assertions.
  • Add deterministic coverage for late contenders reusing an already initialized default client.

@im47cn-bot
im47cn-bot requested a review from im47cn as a code owner October 18, 2026 10:37
@changeset-bot

changeset-bot Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: bd70697

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai

coderabbitai Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 7f6fe1ea-1849-4bae-9220-159f3dbc2abf
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@sourcery-ai

sourcery-ai Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

审阅者指南

本 PR 将 wop-specs 钉版升级至包含 RSA2048 加密向量的提交并同步校验值,新增 RSA2048 签名和 OAEP 解包的字节级黄金测试,同时将配置模板及中英文文档的默认算法套件统一为 WOP-RSA2048-SHA256;互操作向量保持不变。

文件级变更

变更 详情 文件
升级并锁定 wop-specs 加密向量版本,同时保留互操作向量不变。
  • 将 wopSpecsRef 更新至包含 RSA2048 keys、signature 与 OAEP 向量的提交。
  • 同步 crypto-vectors SHA-256 校验值,并维持 interop 校验值不变。
  • 依赖拉取与逐字节 SHA-256 校验继续采用失败即停策略。
wop-sdk-core/pom.xml
补齐 RSA2048 的签名与 OAEP 解包字节级黄金向量覆盖。
  • 新增 RSA2048 PKCS#1 签名测试,校验签名字节、长度、Base64URL 长度及验签结果。
  • 新增 RSA2048 OAEP 解包测试,校验解包后的 DEK 载荷。
  • 使用新增钉版中的 RSA2048 密钥和向量数据进行测试。
wop-sdk-core/src/test/java/com/wanlianyida/wop/crypto/SignatureStrategyVectorTest.java
wop-sdk-core/src/test/java/com/wanlianyida/wop/crypto/KeyEncryptVectorTest.java
统一 SDK 默认算法套件与文档示例为 RSA2048-SHA256。
  • 将配置文件中的默认套件从 RSA3072 调整为 RSA2048。
  • 更新中英文 README 的默认示例及 RSA 签名向量覆盖说明。
config/wopSdkConfigDefault.json
wop-sdk-core/src/main/resources/config/wopSdkConfigDefault.json
README.md
README.en.md

提示与命令

与 Sourcery 交互

  • 触发新的审阅: 在拉取请求中评论 @sourcery-ai review。
  • 继续讨论: 直接回复 Sourcery 的审阅评论。
  • 根据审阅评论创建 GitHub issue: 回复审阅评论,请 Sourcery 根据该评论创建 issue。也可以在审阅评论中回复 @sourcery-ai issue 来创建 issue。
  • 生成拉取请求标题: 在拉取请求标题中的任意位置写入 @sourcery-ai,即可随时生成标题。也可以在拉取请求中评论 @sourcery-ai title,随时重新生成标题。
  • 生成拉取请求摘要: 在拉取请求正文中需要插入摘要的位置写入 @sourcery-ai summary,即可随时生成 PR 摘要。也可以在拉取请求中评论 @sourcery-ai summary,随时重新生成摘要。
  • 生成审阅者指南: 在拉取请求中评论 @sourcery-ai guide,即可随时重新生成审阅者指南。
  • 解决所有 Sourcery 评论: 在拉取请求中评论 @sourcery-ai resolve,即可解决所有 Sourcery 评论。如果你已处理完所有评论且不想再看到它们,这个命令会很有用。
  • 忽略所有 Sourcery 审阅: 在拉取请求中评论 @sourcery-ai dismiss,即可忽略所有现有的 Sourcery 审阅。如果你想从头开始进行新一轮审阅,这个命令尤其有用——别忘了评论 @sourcery-ai review 来触发新的审阅!

自定义体验

访问你的控制面板以进行以下操作:

  • 启用或禁用审阅功能,例如 Sourcery 生成的拉取请求摘要、审阅者指南等。
  • 更改审阅语言。
  • 添加、移除或编辑自定义审阅指令。
  • 调整其他审阅设置。

获取帮助

Original review guide in English

Reviewer's Guide

本 PR 将 wop-specs 钉版升级至包含 RSA2048 加密向量的提交并同步校验值,新增 RSA2048 签名和 OAEP 解包的字节级黄金测试,同时将配置模板及中英文文档的默认算法套件统一为 WOP-RSA2048-SHA256;互操作向量保持不变。

File-Level Changes

Change Details Files
升级并锁定 wop-specs 加密向量版本,同时保留互操作向量不变。
  • 将 wopSpecsRef 更新至包含 RSA2048 keys、signature 与 OAEP 向量的提交。
  • 同步 crypto-vectors SHA-256 校验值,并维持 interop 校验值不变。
  • 依赖拉取与逐字节 SHA-256 校验继续采用失败即停策略。
wop-sdk-core/pom.xml
补齐 RSA2048 的签名与 OAEP 解包字节级黄金向量覆盖。
  • 新增 RSA2048 PKCS#1 签名测试,校验签名字节、长度、Base64URL 长度及验签结果。
  • 新增 RSA2048 OAEP 解包测试,校验解包后的 DEK 载荷。
  • 使用新增钉版中的 RSA2048 密钥和向量数据进行测试。
wop-sdk-core/src/test/java/com/wanlianyida/wop/crypto/SignatureStrategyVectorTest.java
wop-sdk-core/src/test/java/com/wanlianyida/wop/crypto/KeyEncryptVectorTest.java
统一 SDK 默认算法套件与文档示例为 RSA2048-SHA256。
  • 将配置文件中的默认套件从 RSA3072 调整为 RSA2048。
  • 更新中英文 README 的默认示例及 RSA 签名向量覆盖说明。
config/wopSdkConfigDefault.json
wop-sdk-core/src/main/resources/config/wopSdkConfigDefault.json
README.md
README.en.md

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

你好——我已审查你的更改,整体看起来很棒!

Sourcery 评估

需要人工审查。 加载默认配置的用户会在签名和密钥封装时静默地选择 RSA-2048,而不是 RSA-3072,因此错误的安全强度选择会影响他们,却不会触发错误或警告。回退更改可以恢复未来的默认值,但无法更改已经使用 RSA-2048 生成的签名或封装密钥。


Sourcery 对开源项目免费——如果你喜欢我们的审查结果,请考虑分享 ✨
Original comment in English

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Needs a human reviewer. Users loading the default configuration will silently select RSA-2048 for signing and key wrapping instead of RSA-3072, so a mistaken security-strength choice affects them without an error or warning. Reverting restores the future default but cannot change signatures or wrapped keys already produced under RSA-2048.


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

竞速测试线程唤醒错开时后醒者走快路径,弧永不达,ubuntu-JDK21 腿
JaCoCo 分支 0.99<1.00 偶发。反射持锁交错确定性命中该弧。
@im47cn
im47cn merged commit eb22b18 into main Oct 18, 2026
17 checks passed
@im47cn
im47cn deleted the fix/rsa2048-vector-pin branch October 18, 2026 12:37
@im47cn-bot
im47cn-bot restored the fix/rsa2048-vector-pin branch October 19, 2026 23:54
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