Skip to content

fix(runtime-sdk): add rpds to rust package entropy context - #277

Merged
ryanking13 merged 1 commit into
cloudflare:mainfrom
zhangx1n:zhangx1n/rpds-entropy
Sep 29, 2026
Merged

ryanking13 merged 1 commit into
cloudflare:mainfrom
zhangx1n:zhangx1n/rpds-entropy

Conversation

@zhangx1n

Copy link
Copy Markdown
Contributor

rpds (pulled in by jsonschema -> referencing) is a Rust extension that seeds a hash map at import, so importing jsonschema at the top level fails snapshot creation with an entropy error.

It shares the "only the first Rust package makes the call" behavior with jiter/tiktoken/cryptography: a separate allow_bad_entropy_calls(1) fails with a leftover call on 3.12-3.14 in either import order, so it goes into rust_package_context like uuid_utils in #151.

rpds (pulled in by jsonschema -> referencing) is a Rust extension that
seeds a HashMap at import time, so importing jsonschema at the top level
aborts the snapshot with an entropy error.
@github-actions

github-actions Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@zhangx1n

Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

github-actions Bot added a commit that referenced this pull request Sep 28, 2026
@zhangx1n
zhangx1n marked this pull request as ready for review September 28, 2026 02:56

@ryanking13 ryanking13 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@ryanking13
ryanking13 merged commit a097b7c into cloudflare:main Sep 29, 2026
17 of 18 checks passed
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