Skip to content

This PR fixes a logical error in data checksum validation. Without this fix, the function returns None instead of True when no checksum is provided, which could cause silent failures in data integrity checks.#16

Open
BryanBradfo wants to merge 1 commit intoramp-kits:mainfrom
BryanBradfo:fix/checksum-return-true

Conversation

@BryanBradfo
Copy link

Purpose

Fixes a bug where checksum_data returns None instead of True when data_checksum is None.

Changes

  • Added return True in the else block of checksum_data to ensure correct validation.

Testing

  • Validated locally by setting data_checksum = None and confirming checksum_data returns True.
  • Ensured existing checksum validation logic remains unaffected.

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.

1 participant

Comments