Skip to content

Reverse search allowlisted requests with tight packing and stop criteria#245

Merged
ibrajer merged 9 commits into
developfrom
update-request-allowlist-search
Sep 30, 2025
Merged

Reverse search allowlisted requests with tight packing and stop criteria#245
ibrajer merged 9 commits into
developfrom
update-request-allowlist-search

Conversation

@ibrajer
Copy link
Copy Markdown
Contributor

@ibrajer ibrajer commented Sep 29, 2025

Added a new function that supports reverse search of allowlisted requests, tight packing per page returned, and disclosing stop flag that will indicate to the caller when it's safe to abort the search.

@ibrajer ibrajer self-assigned this Sep 29, 2025
@ibrajer ibrajer force-pushed the update-request-allowlist-search branch from 8a15505 to 938b8df Compare September 29, 2025 12:22
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 29, 2025

Static analysis results are available

Hey @ibrajer, you can view Slither reports in the job summary here or download them as artifact here.
Please check them before merging and make sure you have addressed all issues.

@ibrajer ibrajer marked this pull request as ready for review September 29, 2025 12:23
@ibrajer ibrajer requested a review from a team as a code owner September 29, 2025 12:23
Comment thread contracts/src/v0.8/workflow/dev/v2/WorkflowRegistry.sol Outdated
Comment thread contracts/src/v0.8/workflow/dev/v2/WorkflowRegistry.sol Outdated
Comment thread contracts/src/v0.8/workflow/dev/v2/WorkflowRegistry.sol Outdated
infiloop2
infiloop2 previously approved these changes Sep 29, 2025
* --wip-- [skip CI]

* Logic and test fixes

* --wip-- [skip CI]

* Fixes failing tests and updated snapshot

---------

Co-authored-by: Iva Brajer <iva.brajer@smartcontract.com>
infiloop2
infiloop2 previously approved these changes Sep 30, 2025
infiloop2
infiloop2 previously approved these changes Sep 30, 2025
Comment thread contracts/src/v0.8/workflow/dev/v2/WorkflowRegistry.sol Outdated
Comment thread contracts/src/v0.8/workflow/dev/v2/WorkflowRegistry.sol Outdated
Comment thread contracts/src/v0.8/workflow/dev/v2/WorkflowRegistry.sol Outdated
DeividasK
DeividasK previously approved these changes Sep 30, 2025
@ibrajer ibrajer dismissed stale reviews from DeividasK and infiloop2 via 1737386 September 30, 2025 09:51
DeividasK
DeividasK previously approved these changes Sep 30, 2025
@ibrajer ibrajer merged commit 353b3b3 into develop Sep 30, 2025
52 checks passed
@ibrajer ibrajer deleted the update-request-allowlist-search branch September 30, 2025 17:01
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.

4 participants