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.

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

Revert "fix headers/body order (#55)" #74

Merged
merged 1 commit into from
Jul 3, 2023
Merged

Revert "fix headers/body order (#55)" #74

merged 1 commit into from
Jul 3, 2023

Commits on Jul 2, 2023

  1. Revert "fix headers/body order (rs#55)"

    This reverts commit e63e657.
    
    The commit above introduced sane-ordering to the terminal output by
    prioritizing headers first and then the response body. But unfortunately
    it introduces a deadlock in cases where we have no headers to print.
    
    Commands like `curlie` or `curlie -h` will hang.
    
    Sometimes it also causes the program to deadlock even when headers are
    present. It probably has something to do with the code failing to
    properly detect where the headers' section ends. But this is difficult
    to reproduce.
    js-everts committed Jul 2, 2023
    Configuration menu
    Copy the full SHA
    d8df3b4 View commit details
    Browse the repository at this point in the history