Giter VIP home page Giter VIP logo

Comments (9)

zfLQ2qx2 avatar zfLQ2qx2 commented on June 10, 2024 1

@jfagoagas Ok, I just wanted to make sure. Looks good then!

from prowler.

jfagoagas avatar jfagoagas commented on June 10, 2024

Hi @zfLQ2qx2 I really appreciate your feedback but I'm not getting it. Which output format are you referring to? Both CSV and JSON format contains the same information as in v3, we've just standardize the structure using the same for all the providers, including the check's metadata and all the information retrieved from the Cloud provider.

Please, give us more details about what you are missing. Here https://docs.prowler.com/projects/prowler-open-source/en/latest/tutorials/reporting/#v4-deprecations you can see the changes from v3.

Which version of Prowler are you running?

Thanks for using Prowler 🚀

from prowler.

zfLQ2qx2 avatar zfLQ2qx2 commented on June 10, 2024

@jfagoagas It looks like the two main issues are that 1) there is no equivalent for the "Compliance" member of the Prowler v3 json formation in the v4 oscf json format and 2) there is no equivalent for the "CheckID" member of the Prowler v3 json formation in the v4 oscf json format. The documentation mentions metadata.event_code being the v4 equivalent for CheckID but that is not actually present in the ocsf results.

from prowler.

jfagoagas avatar jfagoagas commented on June 10, 2024

@zfLQ2qx2 both of your requests will be pushed in the next Prowler release, right now are merged on master branch. You can give this a try there.

Thanks for using Prowler 🚀

from prowler.

zfLQ2qx2 avatar zfLQ2qx2 commented on June 10, 2024

@jfagoagas Thank you, I'll be able to try it this evening and let you know

from prowler.

zfLQ2qx2 avatar zfLQ2qx2 commented on June 10, 2024

@jfagoagas I've had a look:

    "unmapped": {
        "check_type": "IAM",
        "related_url": "https://docs.aws.amazon.com/IAM/latest/UserGuide/what-is-access-analyzer.html",
        "categories": "",
        "depends_on": "",
        "related_to": "",
        "notes": "",
        "compliance": {
            "CIS-1.5": [
                "1.20"
            ],
            "CIS-1.4": [
                "1.20"
            ],
            "CIS-3.0": [
                "1.20"
            ],
            "AWS-Account-Security-Onboarding": [
                "Enabled security services",
                "Create analyzers in each active regions",
                "Verify that events are present in SecurityHub aggregated view"
            ],
            "CIS-2.0": [
                "1.20"
            ]
        }
    },

I'm assuming that "unmapped" is the "ocsf" way of passing information outside of the "ocsf" standard. The compliannce."AWS-Account-Security-Onboarding" does not look right - maybe namespace collision in the unmapped members and multiple 'compliance' objects are being merged?

from prowler.

jfagoagas avatar jfagoagas commented on June 10, 2024

Hi @zfLQ2qx2 as you said the unmapped section in the OCSF schema allows you to include information generated from Prowler that are not currently mapped in the Detection Finding event, which is the event we use to generate findings.

Regarding the compliance format, it's fine since we gather the requirements and each compliance framework has their specific format and naming.

from prowler.

zfLQ2qx2 avatar zfLQ2qx2 commented on June 10, 2024

@jfagoagas If I can re-open this - I found one other omission, in Prowler v3 the ResourceDetails member of the json format was being used to return the name of AWS security groups as this is an attribute of the resource and not a tag, and the sg-xxxxxxxxx identifier isn't really useful for whitelisting. In the Prowler v4 format ResourceDetails is unmapped though currently does not appear in the unmapped section. I'm not aware of any other resource like this, I think it is a hold-over from AWS's earliest days, tagging was an afterthought, each AWS service has its own tagging API, and they don't all work the same way! Though they all seem to follow one of seven different patterns. Crazy stuff.

from prowler.

jfagoagas avatar jfagoagas commented on June 10, 2024

HI @zfLQ2qx2 you were right, we've just included that in the above PR. We'll include that in the next Prowler release. Thanks!

from prowler.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.