-
Notifications
You must be signed in to change notification settings - Fork 32
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
193 resolve docs issues #195
Conversation
Codecov Report
@@ Coverage Diff @@
## main #195 +/- ##
==========================================
- Coverage 72.75% 70.03% -2.73%
==========================================
Files 83 83
Lines 4695 4655 -40
==========================================
- Hits 3416 3260 -156
- Misses 1194 1310 +116
Partials 85 85
... and 10 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
docs/_docs/user-guide/eldritch.md
Outdated
@@ -223,10 +228,12 @@ The <b>pivot.arp_scan</b> method is being proposed to allow users to enumerate h | |||
The <b>pivot.port_forward</b> method is being proposed to providde socat like functionality by forwarding traffic from a port on a local machine to a port on a different machine allowing traffic to be relayed. | |||
|
|||
### pivot.ncat | |||
`pivot.ncat(address: str, port: int, data: str, str: portocol ) -> str` | |||
`pivot.ncat(address: str, port: int, data: str, portocol: str ) -> str` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
portocol, or protocol?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or is this intentional :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😅 just wish I knew how to spell.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My old requested changes aren't valid, unless we're also fixing the spelling in code:
Yep haven't implemented those changes yet. |
* Updated docs. * Added golem to toc. * Fix typo start to sort by name. * Add assets docs. * Unified wording. * Clean up spelling.
* sysshell now return dict with stdout stderr status * Expanded sys exec output. * Update tests.yml (#207) * Update tests.yml needed * Update tests.yml ugg windows * Update tests.yml fml * 182 eldritch sleep blocks imix agent (#208) * Couple tests. * Resolve async errors. * 193 resolve docs issues (#195) * Updated docs. * Added golem to toc. * Fix typo start to sort by name. * Add assets docs. * Unified wording. * Clean up spelling. * Update tests. * Removed double defined test. --------- Co-authored-by: Nicholas O'Brien <[email protected]>
194 expand shell command to return exit code and stderr (#209) * sysshell now return dict with stdout stderr status * Expanded sys exec output. * Update tests.yml (#207) * Update tests.yml needed * Update tests.yml ugg windows * Update tests.yml fml * 182 eldritch sleep blocks imix agent (#208) * Couple tests. * Resolve async errors. * 193 resolve docs issues (#195) * Updated docs. * Added golem to toc. * Fix typo start to sort by name. * Add assets docs. * Unified wording. * Clean up spelling. * Update tests. * Removed double defined test. --------- Co-authored-by: Nicholas O'Brien <[email protected]>
Adds golem to toc and fixes doc typos.