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

Update tools.json #1032

Merged
merged 6 commits into from
Jan 2, 2025
Merged

Update tools.json #1032

merged 6 commits into from
Jan 2, 2025

Conversation

ctflearner
Copy link
Contributor

Added the Tool for Various Platform.

Copy link

github-actions bot commented Jan 1, 2025

The following issues were identified, when validating against the schema:

Summary (click the triangle/control to the left to expand)
_data/tools.json invalid
data/3/license must be equal to one of the allowed values, data/3/type must be equal to one of the allowed values, data/6/type must be equal to one of the allowed values, data/7/type must be equal to one of the allowed values, data/8/type must be equal to one of the allowed values, data/9/type must be equal to one of the allowed values

Try to fix the Error
Copy link

github-actions bot commented Jan 1, 2025

The following issues were identified, when validating against the schema:

Summary (click the triangle/control to the left to expand)
_data/tools.json invalid
data/3/license must be equal to one of the allowed values, data/3/type must be equal to one of the allowed values, data/6/type must be equal to one of the allowed values, data/7/type must be equal to one of the allowed values, data/8/type must be equal to one of the allowed values, data/9/type must be equal to one of the allowed values

@kingthorin
Copy link
Contributor

kingthorin commented Jan 1, 2025

Please lookup SAST and DAST, then set the type appropriately. A single tool cannot be (or is highly highly unlikely to be) simultaneously one or the other.

@ctflearner
Copy link
Contributor Author

Ok I'm Correcting it

Copy link

github-actions bot commented Jan 1, 2025

The following issues were identified, when validating against the schema:

Summary (click the triangle/control to the left to expand)
_data/tools.json invalid
data/3/license must be equal to one of the allowed values

@ctflearner
Copy link
Contributor Author

I have made the correction but still it shows that the schema-check is failed??

@kingthorin
Copy link
Contributor

One of the licenses is an unexpected value. If you click the schema link in the comment it'll show you the accepted values.

Copy link

github-actions bot commented Jan 1, 2025

The following issues were identified, when validating against the schema:

Summary (click the triangle/control to the left to expand)
_data/tools.json invalid
data/3/license must be equal to one of the allowed values

Copy link

github-actions bot commented Jan 1, 2025

The following issues were identified, when validating against the schema:

Summary (click the triangle/control to the left to expand)
_data/tools.json invalid
data/2/license must be equal to one of the allowed values, data/3/license must be equal to one of the allowed values

@kingthorin
Copy link
Contributor

Thanks. I'll check a few things in the morning and approve/merge.

Happy New Year 🎉

@ctflearner
Copy link
Contributor Author

Thanks. I'll check a few things in the morning and approve/merge.

Happy New Year 🎉

Thanks , Happy New Year to you also

@kingthorin kingthorin merged commit 48a6853 into OWASP:master Jan 2, 2025
1 check passed
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.

2 participants