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.

Sign up for GitHub

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

Jump to bottom

Don't let Uri to escape query #2208

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from
Open

Don't let Uri to escape query #2208

wants to merge 6 commits into from
+269 −264

Commits on May 27, 2024

  1. Force not escape query

    @alexeyzimarev
    alexeyzimarev committed May 27, 2024
    Configuration menu
    Copy the full SHA
    25c4124 View commit details
    Browse the repository at this point in the history

  2. Fix the test

    @alexeyzimarev
    alexeyzimarev committed May 27, 2024
    Configuration menu
    Copy the full SHA
    27bec28 View commit details
    Browse the repository at this point in the history

  3. Fix the URL builder tests

    @alexeyzimarev
    alexeyzimarev committed May 27, 2024
    Configuration menu
    Copy the full SHA
    af14a20 View commit details
    Browse the repository at this point in the history

  4. Encode the resource query as it should be.

    @alexeyzimarev
    alexeyzimarev committed May 27, 2024
    Configuration menu
    Copy the full SHA
    41a2ba4 View commit details
    Browse the repository at this point in the history

  5. Some more tweaks for query encoding but it still doesn't seem to work… 

    … on .NET 4.8
    @alexeyzimarev
    alexeyzimarev committed May 27, 2024
    Configuration menu
    Copy the full SHA
    af7d572 View commit details
    Browse the repository at this point in the history

  6. Ok, it is dontEscape so needs to be true.

    @alexeyzimarev
    alexeyzimarev committed May 27, 2024
    Configuration menu
    Copy the full SHA
    04699ff View commit details
    Browse the repository at this point in the history