Git Product home page Git Product logo

Comments (5)

ethanb-cisa avatar ethanb-cisa commented on May 24, 2024

Thanks for reporting this. We will look into it how to fix.

from scubagear.

schrolla avatar schrolla commented on May 24, 2024

Need to verify if the assumption that all policies are active and the result is the aggregated logical AND of all those policies or not.

from scubagear.

skirkpatrickms avatar skirkpatrickms commented on May 24, 2024

from scubagear.

tkol2022 avatar tkol2022 commented on May 24, 2024

Reviewed 12/14

@schrolla @buidav Please review and close if this is OBE. If it is still valid, please check to ensure the size and priority labels are accurate.

from scubagear.

schrolla avatar schrolla commented on May 24, 2024

While the DLP capability allows for processing and evaluating multiple policies in the specified order, the baseline states that "a custom policy" shall be configured. So the requirement being assessed is that a single custom policy contains one or more rules with ALL sensitive info types present. The bug stated above, as I understand it, and please correct me if I'm wrong, is that the test scenario splits the sensitive info types across two different policies. In the case above, I'd evaluate the assessment result as a fail on MS.DEFENDER.4.1v1 as neither has a set of rules that contain the 3 minimum info types (SSN, PII, TIN).
The Defender code was updated for the upcoming release which improved the evaluation logic. I ran a test by creating two custom policies as described above: 1 targeting Teams with SSN, CC, and Passport and a second targeting Devices with only CC. Neither policy included ITIN and these were the only two policies set to On. I ran the test in a G5 tenant. The result was a fail, as expected since no policy contains the required ITIN (which is identified as the missing type in the fail report details for MS.DEFENDER.4.1v1). Also note, that since no single custom policy meets 4.1 the ancillary baselines for MS.DEFENDER.4.2-4.4 also fail since they check additional settings associated with the required info type rules.

Screenshot 2023-12-15 at 8 47 14 AM

As such, I believe the upcoming release addresses this issue by resolving the false positive and that this item can be closed.

from scubagear.

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.