Skip to content
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.

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

feat: Make payload size limit a shardus config #15

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

jintukumardas
Copy link

Added the payload size limit to the SnOpts type and updated the Sn function to use the provided limit.

Copy link

PR Reviewer Guide 🔍

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪
🧪 PR contains tests
🔒 No security concerns identified
⚡ Key issues to review

Error Handling
The error handling for the conversion of JavaScript number to Rust usize (line 420) is marked with a TODO but not implemented. This could lead to potential runtime errors if the conversion fails or is inaccurate.

Error Handling
The error message in line 102 does not provide details about what action to take or more context about the error, which might not be helpful for debugging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant