Skip to content
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

Support for PowerShell CLM #4923

Merged
merged 26 commits into from
Aug 8, 2024
Merged

Conversation

DenisNikulin5
Copy link
Contributor

This PR is a copy of #4502 with minor changes

Pull request for support of Constrained Language Mode for the ADO pipeline agent.
This refactors the code out of the CLI and into a script that can be signed, verified and worked with using WDAC and CLM.

Before we used a code of block inside PowerShell3Handler which was not allowed in constrained language mode, and it was a root cause of this error #4492

Testing: manually

Tim Brigham and others added 24 commits July 25, 2024 17:22
(cherry picked from commit be7847e)
commit b0d2f96cfc6d9d15aae190071260b2c605d812ff
Author: Tim Brigham <[email protected]>
Date:   Mon Nov 13 16:44:37 2023 -0500

    Updates to displayed notes

commit 57c35991758ad5edc75c20d99f6e6dc4fde615c6
Author: Tim Brigham <[email protected]>
Date:   Mon Nov 13 16:19:32 2023 -0500

    Documentation for the pipeline if CLM is enabled.

commit 08e7a390d49a074cf12ec0f4506c833082f5edef
Author: Tim Brigham <[email protected]>
Date:   Fri Nov 10 17:14:21 2023 -0500

    Description

commit 6a4672ae7c45ad61fe866957d9e1c70d1a736684
Author: Tim Brigham <[email protected]>
Date:   Fri Nov 10 16:20:13 2023 -0500

    Improvements in verbose log mesages.

(cherry picked from commit d92ceb8)
@DenisNikulin5 DenisNikulin5 merged commit f9260a0 into master Aug 8, 2024
12 checks passed
@DenisNikulin5 DenisNikulin5 deleted the users/v-denikulin/clm-enforcement branch August 8, 2024 13:38
@timbrigham-oc
Copy link

Many thanks for addressing getting this merged. This issue has been a sore spot for me for years!

@DenisNikulin5
Copy link
Contributor Author

@timbrigham-oc Thanks for your contribution! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants