Open
Conversation
Added sections on Ownership, Result, and Macros in Rust.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Updated the Selenium section to include SeleniumBase with stealth capabilities and modes.
There was a problem hiding this comment.
Pull request overview
Updates mdBook documentation to expand the Rust basics notes and revise the Cloudflare scraping guidance with an additional headless-browser automation option.
Changes:
- Add new Rust Basics sections covering ownership,
Result, and code coverage withcargo-tarpaulin. - Update “Fortified Headless Browsers” guidance to replace the Selenium entry with a SeleniumBase-based recommendation and fix a typo (“detected”).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
src/todo/rust-basics.md |
Adds introductory Rust content (ownership, Result) and a tarpaulin code coverage snippet. |
src/network-services-pentesting/pentesting-web/uncovering-cloudflare.md |
Fixes wording/formatting and updates Selenium tooling guidance in the Cloudflare scraping section. |
Comments suppressed due to low confidence (5)
src/todo/rust-basics.md:17
- In the code comment, the block delimiter is described as “brackets”, but Rust blocks are delimited by braces (
{}); this is misleading for readers. Update the wording to “braces/curly braces” (and consider splitting the long inline comment into a preceding line comment for readability).
{ // Scope of a variable is within the block it is declared in, which is denoted by brackets
let teacher_age: u32 = 41;
src/todo/rust-basics.md:8
- The PR description/title focuses on adding a Cloudflare-bypass library, but this file also adds new Rust tutorial content (ownership/result/tarpaulin). Consider splitting the Rust docs updates into a separate PR or updating the PR description so reviewers can assess scope accurately.
### Ownership of variables
Memory is managed through a system of ownership with the following rules that the compiler checks at compile time:
src/todo/rust-basics.md:412
cargo binstall cargo-tarpaulinrequirescargo-binstallto be installed first; as written, the snippet is likely to fail for readers on a fresh Rust setup. Consider either usingcargo install cargo-tarpaulin(consistent with the rest of this section) or explicitly noting the prerequisite and providing both installation options.
```bash
cargo binstall cargo-tarpaulin
cargo tarpaulin # no options are required, if no root directory is defined Tarpaulin will run in the current working directory.
src/network-services-pentesting/pentesting-web/uncovering-cloudflare.md:118
- The SeleniumBase bullet makes strong claims like “bypass bot detection” and “solve CAPTCHAs”. To avoid overstating capabilities and encouraging misuse, consider rephrasing more neutrally (e.g., “includes anti-detection features”) and add a brief note that this should only be used for authorized testing/scraping.
- **Selenium:** [SeleniumBase](https://github.com/seleniumbase/SeleniumBase) is a modern browser automation framework featuring built-in stealth capabilities. It offers two modes: **UC Mode**, an optimized Selenium ChromeDriver patch based on [undetected-chromedriver](https://github.com/ultrafunkamsterdam/undetected-chromedriver), and **CDP Mode**, which can bypass bot detection, solve CAPTCHAs, and leverage advanced methods from the Chrome DevTools Protocol.
src/network-services-pentesting/pentesting-web/uncovering-cloudflare.md:119
- Line 119 contains only whitespace (likely intended as a blank line). Prefer an actual empty line to avoid trailing-whitespace noise and potential markdown lint/build differences.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
add a new modern library help bypass cloudflare detection