Skip to content

check nuget before runasuser#461

Merged
jworkmanjc merged 4 commits intomasterfrom
SA-3135_runAsUserCommandUpdate
Jan 26, 2023
Merged

check nuget before runasuser#461
jworkmanjc merged 4 commits intomasterfrom
SA-3135_runAsUserCommandUpdate

Conversation

@jworkmanjc
Copy link
Copy Markdown
Contributor

@jworkmanjc jworkmanjc commented Jan 25, 2023

Issues

  • SA-3135 - Check for Nuget before installing runAsUser

What does this solve?

Before installing a module Nuget must be installed on a windows device. This is typically installed at some point but not by default.

I also bumped the net-ldap version due to a dependabot issue raised this week

Is there anything particularly tricky?

How should this be tested?

On a new system or a system without Nuget/ runasuser installed (instructions to uninstall nuget) run the command changed in this PR, the command should install Nuget, RunAsUser and execute the code through the JumpCloud PowerShell Module

Screenshots

Copy link
Copy Markdown
Contributor

@gweinjc gweinjc left a comment

Choose a reason for hiding this comment

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

Worked on my windows VM 👍

Copy link
Copy Markdown
Contributor

@kmaranionjc kmaranionjc left a comment

Choose a reason for hiding this comment

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

Worked, successfully installed

@jworkmanjc jworkmanjc merged commit 5218306 into master Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants