Update System.Net.HttpWebRequest.cs#473
Conversation
It seems like this comment was forgotten. .AbsolutePath does not include query string whis is getting lost during request
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
@dotnet-policy-service agree |
Ellerbach
left a comment
There was a problem hiding this comment.
Thanks, indeed, that will work better! Thanks!
|
@DmytroKoval thank you again for your contribution! 🙏😄 .NET nanoFramework is all about community involvement, and no contribution is too small. Please edit it and add an entry with your GitHub username in the appropriate location (names are sorted alphabetically): (Feel free to adjust your name if it's not correct) |
Description
I tried to do a simple request to url which contains query string (part of the URL after "?"), but query string did not reach server.
It seems like this comment was forgotten.
.AbsolutePathdoes not include query string which is getting lost during request.How Has This Been Tested?
Screenshots
Types of changes
Checklist: