Skip to content

Expand webhook-utils signature edge case tests - #268

Merged
Lakes41 merged 1 commit into
Adamantine-guild:mainfrom
silentgeckoaudit3801:test/webhook-utils-signature-edges-256
Jul 23, 2026
Merged

Expand webhook-utils signature edge case tests#268
Lakes41 merged 1 commit into
Adamantine-guild:mainfrom
silentgeckoaudit3801:test/webhook-utils-signature-edges-256

Conversation

@silentgeckoaudit3801

Copy link
Copy Markdown
Contributor

Summary

  • add webhook signature edge-case coverage for future timestamps inside tolerance
  • cover empty �1, reordered headers with extra metadata, and same-length wrong signature rejection
  • add a static smoke test that the verifier keeps using imingSafeEqual after the length check

Testing

  • Not run locally; this workspace is intentionally avoiding project dependency/toolchain execution.

@Lakes41
Lakes41 merged commit 3a02f0c into Adamantine-guild:main Jul 23, 2026
1 check passed
@silentgeckoaudit3801

Copy link
Copy Markdown
Contributor Author

Thanks for reviewing and merging this. PR #267 for issue #253 was also merged in the same campaign window.

Is there an official GrantFox/reward claim or payment step I should complete for these merged Maybe Rewarded issues (#253 and #256)? I want to follow the documented process and avoid posting duplicate claim noise.

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