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

QA Report #17

Open
c4-bot-2 opened this issue Sep 18, 2024 · 3 comments
Open

QA Report #17

c4-bot-2 opened this issue Sep 18, 2024 · 3 comments
Labels
bug Something isn't working grade-a Q-20 QA (Quality Assurance) Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax sufficient quality report This report is of sufficient quality

Comments

@c4-bot-2
Copy link
Contributor

See the markdown file with the details of this report here.
@c4-bot-2 c4-bot-2 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 Sep 18, 2024
c4-bot-10 added a commit that referenced this issue Sep 18, 2024
@c4-bot-10
falconhoof issue #17
4e0a130
c4-bot-7 added a commit that referenced this issue Sep 18, 2024
@c4-bot-7
falconhoof data for issue #17
7bf84ed
@howlbot-integration howlbot-integration bot added the sufficient quality report This report is of sufficient quality label Sep 20, 2024
@Qormatic
Copy link

Qormatic commented Oct 11, 2024 •
edited
Loading

Hi @3docSec ,

I don't think this QA report has been reviewed in case you missed it.

Many thanks!

@3docSec
Copy link

3docSec commented Oct 14, 2024

Thanks for flagging, I did miss this one too indeed.

L-01 dup with #11, with full credit because it identifies the DoS attack path of invalidating credentials. This one I didn't miss 😉
L-02 NC (just about naming)
L-03 missing validation - OK
L-04 tokens with high decimals are OOS as per readme -> Invalid
L-05 OK-ish

@c4-judge
Copy link
Contributor

3docSec marked the issue as grade-a

@c4-judge c4-judge mentioned this issue Oct 14, 2024
Closed
@C4-Staff C4-Staff added the Q-20 label Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working grade-a Q-20 QA (Quality Assurance) Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax sufficient quality report This report is of sufficient quality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@Qormatic @c4-judge @C4-Staff @c4-bot-2 @3docSec