René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"WebCmdlets SetRequestContent wrong logic?","articleBody":"### Prerequisites\n\n- [X] Write a descriptive title.\n- [X] Make sure you are able to repro it on the [latest released version](https://github.com/PowerShell/PowerShell/releases)\n- [X] Search the existing issues.\n- [X] Refer to the [FAQ](https://github.com/PowerShell/PowerShell/blob/master/docs/FAQ.md).\n- [X] Refer to [Differences between Windows PowerShell 5.1 and PowerShell](https://docs.microsoft.com/powershell/scripting/whats-new/differences-from-windows-powershell).\n\n### Steps to reproduce\n\nI was cleaning up the code of WebCmdlets #18964 and I noticed some code that seems incorrect. (WebRequestPSCmdlet.Common.cs lines 1114-1134)\r\n\r\n```cs\r\nif (content is FormObject form)\r\n                {\r\n                    SetRequestContent(request, form.Fields);\r\n                }\r\n                else if (content is IDictionary dictionary \u0026\u0026 request.Method != HttpMethod.Get)\r\n                {\r\n                    SetRequestContent(request, dictionary);\r\n                }\r\n                else if (content is XmlNode xmlNode)\r\n                {\r\n                    SetRequestContent(request, xmlNode);\r\n                }\r\n                else if (content is Stream stream)\r\n                {\r\n                    SetRequestContent(request, stream);\r\n                }\r\n                else if (content is byte[] bytes)\r\n                {\r\n                    SetRequestContent(request, bytes);\r\n                }\r\n                else if (content is MultipartFormDataContent multipartFormDataContent)\r\n```\r\n\n\n### Expected behavior\n\n```console\n`else if (content is IDictionary dictionary \u0026\u0026 request.Method != HttpMethod.Get)`\r\n\r\nIt seems this line doesn't set the request.Content if the method is GET and the body is a dictionary\n```\n\n\n### Actual behavior\n\n```console\nI think we should have a consistent behaviour for other type of bodys, for form and infile\r\n\r\nShould we always set the request.Content, or never do it?\n```\n\n\n### Error details\n\n_No response_\n\n### Environment data\n\n```powershell\nName                           Value\r\n----                           -----\r\nPSVersion                      7.3.1\r\nPSEdition                      Core\r\nGitCommitId                    7.3.1\r\nOS                             Microsoft Windows 10.0.19045\r\nPlatform                       Win32NT\r\nPSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}\r\nPSRemotingProtocolVersion      2.3\r\nSerializationVersion           1.1.0.1\r\nWSManStackVersion              3.0\n```\n\n\n### Visuals\n\n_No response_","author":{"url":"https://github.com/CarloToso","@type":"Person","name":"CarloToso"},"datePublished":"2023-01-19T00:54:35.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":1},"url":"https://github.com/18973/PowerShell/issues/18973"}