FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add null host name check in WSMan by iSazonov · Pull Request #11288 · PowerShell/PowerShell · GitHub

Add null host name check in WSMan - #11288

Merged
Andrew (anmenaga) merged 3 commits into
PowerShell:masterfrom
iSazonov:fix-null-in-wsman
Dec 9, 2019
Merged

Add null host name check in WSMan#11288
Andrew (anmenaga) merged 3 commits into
PowerShell:masterfrom
iSazonov:fix-null-in-wsman

Conversation

Ilya (iSazonov) commented Dec 7, 2019
edited by anmenaga
Loading

Copy link
Copy Markdown
Collaborator

PR Summary

Add null check for host name before path normalization.

PR Context

Fix #11278

PR Checklist

Comment thread tools/releaseBuild/setReleaseTag.ps1 Outdated
ghost added Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept and removed Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept labels Dec 7, 2019

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

Copy link
Copy Markdown

CodeFactor issues are false positives.

Andrew (anmenaga) merged commit 12bf530 into PowerShell:master Dec 9, 2019
Ilya (iSazonov) deleted the fix-null-in-wsman branch December 9, 2019 19:16

Copy link
Copy Markdown

🎉v7.0.0-rc.2 has been released which incorporates this pull request.:tada:

Handy links:

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

WSMan provider throws exception for non-existent path with -PathType Container

7 participants


Back | FazBrowse Home | New Git URL