Skip to content

DAT-5, S1: Add Length validation #9

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.

Sign up for GitHub

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

Jump to bottom
Merged
yoshidan merged 3 commits into from
Jun 30, 2025
Merged

DAT-5, S1: Add Length validation #9

yoshidan merged 3 commits into from
Jun 30, 2025
+96 −20

Conversation

@yoshidan
Copy link
Collaborator

@yoshidan yoshidan commented Jun 18, 2025 •
edited
Loading

No description provided.

@yoshidan yoshidan changed the title Fixed acquisition timing of field_element_index DATA-5: Add Length validation / Fixed acquisition timing of field_element_index Jun 22, 2025
@yoshidan yoshidan marked this pull request as draft June 22, 2025 23:35
@yoshidan yoshidan changed the title DATA-5: Add Length validation / Fixed acquisition timing of field_element_index DAT-5: Add Length validation / Fixed acquisition timing of field_element_index Jun 23, 2025
@yoshidan yoshidan force-pushed the fix/verify_blob_index branch from 37f6344 to 0d34c98 Compare June 23, 2025 09:55
@yoshidan yoshidan marked this pull request as ready for review June 23, 2025 09:55
@yoshidan yoshidan changed the title DAT-5: Add Length validation / Fixed acquisition timing of field_element_index DAT-5, S1: Add Length validation / Fixed acquisition timing of field_element_index Jun 23, 2025
@yoshidan yoshidan requested a review from siburu June 26, 2025 10:35
@yoshidan yoshidan force-pushed the fix/verify_blob_index branch from 0d34c98 to fbd3b5b Compare June 28, 2025 02:08
@yoshidan yoshidan changed the title DAT-5, S1: Add Length validation / Fixed acquisition timing of field_element_index DAT-5, S1: Add Length validation Jun 28, 2025
siburu
siburu approved these changes Jun 29, 2025
View reviewed changes
Copy link
Collaborator

@siburu siburu left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

Naohiro Yoshida added 2 commits June 30, 2025 10:09
@yoshidan yoshidan merged commit cb550f2 into main Jun 30, 2025
1 check passed
@yoshidan yoshidan deleted the fix/verify_blob_index branch September 4, 2025 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@siburu siburu siburu approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yoshidan @siburu