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

Fix: Remove recipients from the array #184

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Fix: Remove recipients from the array #184

wants to merge 4 commits into from

Conversation

ogarciarevett
Copy link
Collaborator

@ogarciarevett ogarciarevett self-assigned this Jan 15, 2025
@octane-security-app Octane Security App
Copy link

octane-security-app bot commented Jan 15, 2025 •
edited
Loading

Summary by Octane

New Contracts

No new contracts were added in this PR.

Updated Contracts

  • PaymentRouterNative.sol: Updated the removal of fee recipients by also deleting them from the array; removed batch fee recipient update functionality.

🔗 Commit Hash: 6b529d7

@octane-security-app Octane Security App
Copy link

octane-security-app bot commented Jan 15, 2025 •
edited
Loading

Overview

Octane AI analysis has finished. No vulnerabilities were found. Cheers! 🎉🎉🎉

🔗 Commit Hash: 6b529d7

@openzeppelin-code OpenZeppelin Code
Copy link

openzeppelin-code bot commented Jan 15, 2025 •
edited
Loading

Fix: Remove recipients from the array

Generated at commit: 6b529d7cd4014cb34ca72cd39fd87db0820e9641

🚨 Report Summary

Severity Level Results
Contracts Critical
High
Medium
Low
Note
Total		 2
4
0
11
41
58
Dependencies Critical
High
Medium
Low
Note
Total		 1
0
0
0
0
1

For more details view the full report in OpenZeppelin Code Inspector

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vasinl124 vasinl124 Awaiting requested review from vasinl124 vasinl124 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@ogarciarevett ogarciarevett

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ogarciarevett