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

Call VDP_Close() for all cases and revert regression #4068

Closed
wants to merge 3 commits into from
Closed

Call VDP_Close() for all cases and revert regression #4068

wants to merge 3 commits into from

Commits on Feb 28, 2024

  1. In transport delivery functions, close VDPs always 

    Part 1 of the fix for varnishcache#4067
    @nigoroll
    nigoroll committed Feb 28, 2024
    Configuration menu
    Copy the full SHA
    66e5ee8 View commit details
    Browse the repository at this point in the history

  2. Partly revert "Close VDPs for error conditions in cnt_transmit()" 

    This reverts the code change from commit 6d423aa,
keeps the test case and adds a VDP_Close() for the case that the
transport deliver function is not called.

Part 2 of the fix for varnishcache#4067

Conflicts:
	bin/varnishd/cache/cache_req_fsm.c
    @nigoroll
    nigoroll committed Feb 28, 2024
    Configuration menu
    Copy the full SHA
    498d1ce View commit details
    Browse the repository at this point in the history

  3. Ensure that transports call VDP_Close()

    @nigoroll
    nigoroll committed Feb 28, 2024
    Configuration menu
    Copy the full SHA
    1ab5713 View commit details
    Browse the repository at this point in the history