Skip to content

fix(docs): resolve high-severity contradictions and wrong links from the review - #1172

Closed
mazzma12 wants to merge 2 commits into
mainfrom
fix/docs-review-contradictions
Closed

mazzma12 wants to merge 2 commits into
mainfrom
fix/docs-review-contradictions

Conversation

@mazzma12

Copy link
Copy Markdown
Contributor

Fixes the 61 high and 35 cross-section contradictions and 55 wrong-target links from the docs review, mirrored to versioned copies where still true.
Also updates redirected hub/golang URLs, adds placeholder logos for 3 missing images, and removes the obsolete Console troubleshooting page (redirected to Stack Health).
To check: CTI "22M+ observed IPs" claim and the haproxy template path (bouncer source not verified).

# 5. Fetch the blocklist (using integration credentials)
curl 'https://admin.api.crowdsec.net/v1/integrations/INTEGRATION_ID/content' \
-H 'x-api-key: INTEGRATION_API_KEY'
curl -u 'USERNAME:PASSWORD' 'https://admin.api.crowdsec.net/v1/integrations/INTEGRATION_ID/content'

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Exposed secret in crowdsec-docs/unversioned/tracker_api/api_integrations.mdx - low severity
Discovered a potential basic authorization token provided in a curl command, which could compromise the curl accessed resource.

Reply @AikidoSec ignore: [REASON] to ignore this issue.
More Info

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

you are wrong, this is a documentation content

@aws-amplify-eu-west-1

Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-1172.d1to60jd2gb6y6.amplifyapp.com

@mazzma12

Copy link
Copy Markdown
Contributor Author

Split into #1173 (links only, quick read) and #1174 (contradictions, needs checking), both rebased on current main and building clean. Closing this one in favour of those two — reopen if you would rather review it whole.

@mazzma12 mazzma12 closed this Sep 30, 2026
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.

1 participant