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
chore(docs): update BridgeCrew links in metadata to our local docs link #3858
Conversation
You can check the documentation for this PR here -> SaaS Documentation |
You can check the documentation for this PR here -> SaaS Documentation |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3858 +/- ##
==========================================
+ Coverage 85.52% 86.40% +0.88%
==========================================
Files 741 746 +5
Lines 22886 23106 +220
==========================================
+ Hits 19573 19965 +392
+ Misses 3313 3141 -172 ☔ View full report in Codecov by Sentry. |
@@ -249,11 +249,11 @@ Each Prowler check has metadata associated which is stored at the same level of | |||
# Code holds different methods to remediate the FAIL finding | |||
"Code": { | |||
# CLI holds the command in the provider native CLI to remediate it | |||
"CLI": "https://docs.bridgecrew.io/docs/public_8#cli-command", | |||
"CLI": "https://docs.prowler.com/checks/public_8#cli-command", |
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.
I think the link you are referring in this file is https://docs.prowler.com/checks/aws/public-policies/public_8/
tests/lib/check/check_loader_test.py
Outdated
@@ -40,7 +40,7 @@ def get_custom_check_metadata(self): | |||
Remediation=Remediation( | |||
Code=Code( | |||
NativeIaC="", | |||
Terraform="https://docs.bridgecrew.io/docs/bc_aws_s3_20#terraform", | |||
Terraform="https://docs.prowler.com/checks/aws/general-policies/bc_aws_s3_20#terraform", |
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.
This link is not correct, please check it
@@ -46,7 +46,7 @@ def get_custom_check_metadata(self): | |||
Remediation=Remediation( | |||
Code=Code( | |||
NativeIaC="", | |||
Terraform="https://docs.bridgecrew.io/docs/bc_aws_s3_20#terraform", | |||
Terraform="https://docs.prowler.com/checks/aws/general-policies/bc_aws_s3_20#terraform", |
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.
This link is not correct, please check it
@@ -23,7 +23,7 @@ | |||
Code=Code( | |||
NativeIaC="", | |||
Terraform="", | |||
CLI="https://docs.bridgecrew.io/docs/networking_5#cli-command", | |||
CLI="https://docs.prowler.com/checks/networking_5#cli-command", |
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.
This file links isn't working, please check it
Hi @sergargar, I was checking all the links on metadata were well written. This are my findings of not 200 status code please check it:
|
You can check the documentation for this PR here -> SaaS Documentation |
You can check the documentation for this PR here -> SaaS Documentation |
You can check the documentation for this PR here -> SaaS Documentation |
…nk (#3858) Co-authored-by: puchy22 <[email protected]>
Description
Update BridgeCrew links in metadata to our local docs link.
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.