search

dotnet / interactive

.NET Interactive combines the power of .NET with many other languages to create notebooks, REPLs, and embedded coding experiences. Share code, explore data, write, and learn across your apps in ways you couldn't before.
MIT License
2.8k stars 371 forks source link

Changes to request parser to accommodate spaces #3558

Closed bleaphar closed 3 weeks ago

bleaphar commented 1 month ago

This pr contains a major change to the parser to all for spaces in the url of an HttpUrlNode. This pr is a draft until further design considerations are established.

shyamnamboodiripad commented 3 weeks ago

@bleaphar Can we abandon this PR based on the outcome of our discussion offline? I think our conclusion was that allowing whitespace in the grammar could be bug prone and that a better approach may be to introduce a refactoring in the IDE to encode URLs with whitespace correctly instead.