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

Remove CatchAll and body ID manipulation since it should no longer be needed #3987

Merged
merged 3 commits into from Mar 12, 2024
Merged

Remove CatchAll and body ID manipulation since it should no longer be needed #3987

merged 3 commits into from Mar 12, 2024

Commits on Mar 5, 2024

  1. No longer replace the message id since the body will be loaded using … 

    …the unique message id which is a deterministic ID that doesn't contain slashes

Co-authored-by: Mauro Servienti <mauro.servienti@gmail.com>
    @danielmarbach @mauroservienti
    danielmarbach and mauroservienti committed Mar 5, 2024
    Copy the full SHA
    d20d016 View commit details
    Browse the repository at this point in the history

  2. Remove the catch all part because it was only handling /body which sh… 

    …ould not contain slashes anymore

Co-authored-by: Mauro Servienti <mauro.servienti@gmail.com>
    @danielmarbach @mauroservienti
    danielmarbach and mauroservienti committed Mar 5, 2024
    Copy the full SHA
    00e5f99 View commit details
    Browse the repository at this point in the history

  3. Remove the replacement of forward slashes since that could never have… 

    … worked

Co-authored-by: Mauro Servienti <mauro.servienti@gmail.com>
    @danielmarbach @mauroservienti
    danielmarbach and mauroservienti committed Mar 5, 2024
    Copy the full SHA
    cd1edaf View commit details
    Browse the repository at this point in the history