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.

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

OZ audit fixes - Graph Horizon #1125

Open
wants to merge 8 commits into
base: horizon
Choose a base branch
from
Open

OZ audit fixes - Graph Horizon #1125

wants to merge 8 commits into from
+198 −44

Conversation

tmigone
Copy link
Member

@tmigone tmigone commented Mar 7, 2025

No description provided.

@tmigone tmigone changed the title Horizon - solidity changes post audit horizon: post audit solidity changes Mar 7, 2025
@tmigone tmigone mentioned this pull request Mar 7, 2025
Closed
@tmigone tmigone force-pushed the horizon-post-audit-changes branch from 9f37b06 to e2f18af Compare March 7, 2025 18:40
@tmigone tmigone changed the title horizon: post audit solidity changes horizon: audit self reported finding fixes Mar 7, 2025
@tmigone tmigone force-pushed the horizon-post-audit-changes branch from e2f18af to 29285f4 Compare March 7, 2025 18:47
@openzeppelin-code OpenZeppelin Code
Copy link

openzeppelin-code bot commented Mar 7, 2025 •
edited
Loading

OZ audit fixes - Graph Horizon

Generated at commit: d0132874b88f7e5d1929897cefb8546a9e5f48db

🚨 Report Summary

Severity Level Results
Contracts Critical
High
Medium
Low
Note
Total		 2
4
0
15
38
59
Dependencies Critical
High
Medium
Low
Note
Total		 0
0
0
0
0
0

For more details view the full report in OpenZeppelin Code Inspector

@tmigone tmigone requested a review from Maikol March 7, 2025 19:19
juanmardefago and others added 6 commits March 21, 2025 16:27
@juanmardefago @tmigone
fix: add extra parameters to contract events (OZ CR-XX)
222be28
@tmigone
fix: add collection id to rav typehash (OZ CR-01)
5ea383d 
Signed-off-by: Tomás Migone <tomas@edgeandnode.com>
@tmigone
fix: rename misleading custom error name (OZ CR-XX)
64f4d77 
Signed-off-by: Tomás Migone <tomas@edgeandnode.com>
@tmigone
fix: prevent indexer lockout in subgraph service (OZ CR-02)
3447bd9 
Signed-off-by: Tomás Migone <tomas@edgeandnode.com>
@tmigone
fix: limit allowed thawing period range in the subgraph service (OZ M…
f81d8ff 
…-01)

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>
@tmigone
fix: save last parameter staged timestamp on provisions (OZ M-01)
3adab90 
Signed-off-by: Tomás Migone <tomas@edgeandnode.com>
@tmigone tmigone force-pushed the horizon-post-audit-changes branch from 9761f10 to 3adab90 Compare March 21, 2025 19:51
@tmigone
fix: ensure invalid thaw requests are ignored by getThawedTokens (OZ …
2f80dc4 
…L-01)

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>
@tmigone tmigone changed the title horizon: audit self reported finding fixes OZ audit fixes - Graph Horizon Mar 21, 2025
@tmigone
fix: document possible temporary thaw requests blockage when thawing …
d013287 
…period is shortened (OZ L-02)

Signed-off-by: Tomás Migone <tomas@edgeandnode.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Maikol Maikol

Assignees
No one assigned
Labels
horizon solidity
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@tmigone @juanmardefago