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

Wrap inflation errors #500

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Wrap inflation errors #500

wants to merge 1 commit into from

Commits on Feb 23, 2023

  1. feat: wrap inflation errors 

    `pako` has a tendency to throw obscure error strings e.g. "too many length or distance symbols", which can make gracefully handling bad input difficult when an error is thrown from `parseLoginRequest`.
This wraps the inflation step to allow throwing a dedicated error with a static message for inflation issues, while still making the original error available.
    @mastermatt
    mastermatt committed Feb 23, 2023
    Configuration menu
    Copy the full SHA
    67be597 View commit details
    Browse the repository at this point in the history