Bugfix/random pep #636
Bugfix/random pep #636
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #636 +/- ##
===========================================
- Coverage 57.60% 57.58% -0.03%
===========================================
Files 116 116
Lines 3892 3890 -2
===========================================
- Hits 2242 2240 -2
Misses 1650 1650
Continue to review full report at Codecov.
|
| sock.send("start %s\r\n" % (cmdline,)) | ||
| sock.send("net user %s /delete\r\n" % (self._config.user_to_add,)) |
There was a problem hiding this comment.
This change will require testing
There was a problem hiding this comment.
that it's binary? I did a standalone test. socket.send indeed receives binary digits and not string. Or what sort of test did you have in mind?
There was a problem hiding this comment.
Check the exploiter still works
There was a problem hiding this comment.
Problematic, MonkeyZoo doesn't have a test for this
|
@ShayNehmad force pushed the branch and removed the MS08_67 code changes. |
…ernative is using __all__ but that may come with change in functionality.
What is this?
Doesn't fix anything but brings the number of flake8 warnings down to below 100.