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

ManagementConnection - Added GetCimSession method #5

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

FriedrichWeinmann
Copy link
Member

@FriedrichWeinmann FriedrichWeinmann commented Oct 4, 2023

This adds a GetCimSession method to the CM Connection class.

The key reason for this update is to improve interop between the read-only Computer Management component and the external use of Invoke-CimMethod. This enables the transparent passing through of session configurations and policies and will allow commands such as Restart-DbaService to respect proxy settings or protocol preference.

This includes the ability to execute against localhost without the WinRM Service running.

Notes

  • As this only adds a method without changing any existing code, this has no impact on anything (yet)
  • In order for this change to have an effect, it must then later be called from other functions in dbatools. But the library update should go first to avoid breaking tests, rather than updating the functions concurrently.

@potatoqualitee
Copy link
Member

Thank you! 🙏🏼

@potatoqualitee potatoqualitee merged commit ae20f48 into dataplat:main Mar 7, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants