Skip to content

fix(deps): downgrade pkcs8 to v0.10 to match sec1/pkcs1 v0.7 - #10119

Merged
DOsinga merged 1 commit into
aaif-goose:mainfrom
r0x0d:fix-pkcs8-version-conflict
Jun 30, 2026
Merged

fix(deps): downgrade pkcs8 to v0.10 to match sec1/pkcs1 v0.7#10119
DOsinga merged 1 commit into
aaif-goose:mainfrom
r0x0d:fix-pkcs8-version-conflict

Conversation

@r0x0d

@r0x0d r0x0d commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Summary

sec1 v0.7 and pkcs1 v0.7 both pull in pkcs8 v0.10 / der v0.7 transitively. Having pkcs8 v0.11 declared alongside them caused type mismatches (AlgorithmIdentifier, ObjectIdentifier) when building with --no-default-features --features native-tls.

Testing

Ran cargo build --no-default-features --features native-tls and the compilation is successful.

Related Issues

Relates to rhel-lightspeed/goose#43
Discussion: LINK (if any)

Screenshots/Demos (for UX changes)

Before:

After:


This problem was observed during our update to goose 1.38 in Fedora: rhel-lightspeed/goose#43

sec1 v0.7 and pkcs1 v0.7 both pull in pkcs8 v0.10 / der v0.7
transitively. Having pkcs8 v0.11 declared alongside them caused type
mismatches (AlgorithmIdentifier, ObjectIdentifier) when building with
--no-default-features --features native-tls.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@r0x0d

r0x0d commented Jun 30, 2026

Copy link
Copy Markdown
Contributor Author

Should we have an workflow in github actions to test the builds against rustls-tls and native-tls? That way we can prevent such incompatibilities for downstream distributions

@DOsinga DOsinga left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Clean, well-justified fix. pkcs1/sec1 v0.7 pull in pkcs8 v0.10 / der v0.7 transitively, so the v0.11 declaration created an incompatible duplicate type universe in api_client.rs. Downgrading aligns everything. Thanks!

Re your CI suggestion: agree it'd be valuable to have a build check covering both rustls-tls and native-tls to catch this class of downstream breakage — worth a separate issue/PR.

@DOsinga
DOsinga added this pull request to the merge queue Jun 30, 2026
Merged via the queue into aaif-goose:main with commit d056c72 Jun 30, 2026
23 checks passed
@r0x0d
r0x0d deleted the fix-pkcs8-version-conflict branch June 30, 2026 22:02
lifeizhou-ap added a commit that referenced this pull request Jul 1, 2026
* main: (26 commits)
  Fix MCP app sandbox bridge lifecycle (#10064)
  fix(bedrock): send inference config (max_tokens, temperature) on Converse (#9889)
  feat(providers): support OpenRouter request parameters (#9276)
  Migrate local inference model management to ACP (#10124)
  (attempt to) fix disk space errors in linux release builds (#10024)
  feat: add --edit session flag to edit conversation before forking (#9799)
  feat: add iFlytek Spark and Astron MaaS providers (#9837)
  fix(desktop): dedupe Nostr session deep link imports (#9918)
  [codex] Add SessionStart hook parity outside CLI (#9970)
  feat(providers): add Fireworks AI declarative provider (#9990)
  fix(providers): don't retry deterministically-permanent 400s (thinking-block immutability) (#10005)
  fix(deps): downgrade pkcs8 to v0.10 to match sec1/pkcs1 v0.7 (#10119)
  chore(deps): bump actions/cache from 5.0.2 to 6.0.0 (#10051)
  Make OpenAI Responses API store param configurable (#10040)
  remove unsupported model (#10121)
  chore(release): bump version to 1.40.0 (minor) (#10099)
  move ollama provider into goose-providers (#9986)
  UI acp migratoin: Decouple desktop UI types from generated OpenAPI types (#10109)
  fix(otel): use async reqwest client so OTLP export works in `goose serve` mode (#10100)
  feat (acp): exposed available tools in acp schema (#10097)
  ...
lifeizhou-ap added a commit that referenced this pull request Jul 1, 2026
* main: (42 commits)
  Fix MCP app sandbox bridge lifecycle (#10064)
  fix(bedrock): send inference config (max_tokens, temperature) on Converse (#9889)
  feat(providers): support OpenRouter request parameters (#9276)
  Migrate local inference model management to ACP (#10124)
  (attempt to) fix disk space errors in linux release builds (#10024)
  feat: add --edit session flag to edit conversation before forking (#9799)
  feat: add iFlytek Spark and Astron MaaS providers (#9837)
  fix(desktop): dedupe Nostr session deep link imports (#9918)
  [codex] Add SessionStart hook parity outside CLI (#9970)
  feat(providers): add Fireworks AI declarative provider (#9990)
  fix(providers): don't retry deterministically-permanent 400s (thinking-block immutability) (#10005)
  fix(deps): downgrade pkcs8 to v0.10 to match sec1/pkcs1 v0.7 (#10119)
  chore(deps): bump actions/cache from 5.0.2 to 6.0.0 (#10051)
  Make OpenAI Responses API store param configurable (#10040)
  remove unsupported model (#10121)
  chore(release): bump version to 1.40.0 (minor) (#10099)
  move ollama provider into goose-providers (#9986)
  UI acp migratoin: Decouple desktop UI types from generated OpenAPI types (#10109)
  fix(otel): use async reqwest client so OTLP export works in `goose serve` mode (#10100)
  feat (acp): exposed available tools in acp schema (#10097)
  ...
lifeizhou-ap added a commit that referenced this pull request Jul 1, 2026
* main: (31 commits)
  test: generic validator for declarative providers (#10010)
  UI acp migratoin: Decouple desktop UI types from generated OpenAPI types (Part 2) (#10149)
  Remove MCP sampling support (#10087)
  Support TLS for ACP serve (#10088)
  feat (ui): Migrate dictation local model manager to ACP (#10131)
  Fix MCP app sandbox bridge lifecycle (#10064)
  fix(bedrock): send inference config (max_tokens, temperature) on Converse (#9889)
  feat(providers): support OpenRouter request parameters (#9276)
  Migrate local inference model management to ACP (#10124)
  (attempt to) fix disk space errors in linux release builds (#10024)
  feat: add --edit session flag to edit conversation before forking (#9799)
  feat: add iFlytek Spark and Astron MaaS providers (#9837)
  fix(desktop): dedupe Nostr session deep link imports (#9918)
  [codex] Add SessionStart hook parity outside CLI (#9970)
  feat(providers): add Fireworks AI declarative provider (#9990)
  fix(providers): don't retry deterministically-permanent 400s (thinking-block immutability) (#10005)
  fix(deps): downgrade pkcs8 to v0.10 to match sec1/pkcs1 v0.7 (#10119)
  chore(deps): bump actions/cache from 5.0.2 to 6.0.0 (#10051)
  Make OpenAI Responses API store param configurable (#10040)
  remove unsupported model (#10121)
  ...
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