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

Fix the out of scope issue with swagger option #601

Merged

Conversation

Darkiros
Copy link
Collaborator

Fix the out of scope issue with swagger option.

@Darkiros Darkiros requested a review from bretfourbe June 25, 2024 12:24
@Darkiros Darkiros force-pushed the fix_out_of_scope_swagger_URL branch from e7261fc to 90eee6f Compare June 25, 2024 12:31
Copy link

codecov bot commented Jun 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.40%. Comparing base (f8284cb) to head (199ecd2).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #601      +/-   ##
==========================================
- Coverage   79.52%   79.40%   -0.12%     
==========================================
  Files         122      122              
  Lines       10293    10296       +3     
==========================================
- Hits         8185     8176       -9     
- Misses       2108     2120      +12     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Darkiros Darkiros force-pushed the fix_out_of_scope_swagger_URL branch from 90eee6f to 8e4e432 Compare June 25, 2024 12:51
@@ -299,6 +299,22 @@ async def test_swagger_invalid_url(mock_browse):
mock_browse.assert_called_once()


@pytest.mark.asyncio
@mock.patch("wapitiCore.controller.wapiti.Wapiti.add_start_url")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add mock for attack and browse ?

@Darkiros Darkiros force-pushed the fix_out_of_scope_swagger_URL branch from 8e4e432 to 199ecd2 Compare June 28, 2024 09:10
@bretfourbe bretfourbe merged commit 3a0f685 into wapiti-scanner:master Jun 28, 2024
6 of 7 checks 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