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.

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

QA Report #90

Open
c4-bot-9 opened this issue Mar 2, 2024 · 4 comments
Open

QA Report #90

c4-bot-9 opened this issue Mar 2, 2024 · 4 comments
Labels
bug Something isn't working grade-a high quality report This report is of especially high quality Q-27 QA (Quality Assurance) Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax sponsor acknowledged Technically the issue is correct, but we're not going to resolve it for XYZ reasons

Comments

@c4-bot-9
Copy link
Contributor

c4-bot-9 commented Mar 2, 2024

See the markdown file with the details of this report here.

@c4-bot-9 c4-bot-9 added bug Something isn't working QA (Quality Assurance) Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax labels Mar 2, 2024
c4-bot-5 added a commit that referenced this issue Mar 2, 2024
c4-bot-8 added a commit that referenced this issue Mar 2, 2024
@PaperParachute PaperParachute added the high quality report This report is of especially high quality label Mar 8, 2024
@wildmolasses
Copy link

wildmolasses commented Mar 12, 2024

L-01 disputed; stray transfers are not in scope
L-02 not useful; MEV searcher tweaking is out of scope; searcher can write a smart contract method claiming max allowed if this situation is considered important
L-03 dupe of #205
L-04 Not applicable or useful

NCs: nothing that we see as particularly useful (NC-02 will be addressed based on #388)

Ultimately we are glad to see this report, but don't consider it particularly high quality (nor will we dispute it)

@c4-sponsor
Copy link

wildmolasses (sponsor) acknowledged

@c4-sponsor c4-sponsor added the sponsor acknowledged Technically the issue is correct, but we're not going to resolve it for XYZ reasons label Mar 12, 2024
@c4-judge
Copy link
Contributor

MarioPoneder marked the issue as grade-b

@Al-Qa-qa
Copy link

Al-Qa-qa commented Mar 15, 2024

Hi,

Both issues are invalid, but they got Escalations, so I wanted to comment here to consider these issues and judge them. if something changes for these two issues.

Thanks a lot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working grade-a high quality report This report is of especially high quality Q-27 QA (Quality Assurance) Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax sponsor acknowledged Technically the issue is correct, but we're not going to resolve it for XYZ reasons
Projects
None yet
Development

No branches or pull requests

7 participants