varnish / varnish-modules

Collection of Varnish Cache modules (vmods) by Varnish Software
Other
182 stars 86 forks source link

Varnish master change broke var/test06.vtc #224

Closed gquintard closed 6 months ago

gquintard commented 6 months ago

@nigoroll , it looks like 7d06b8b98350de557568d712ff089d3c961d0618 broke https://github.com/varnish/varnish-modules/blob/master/src/tests/var/test06.vtc

From what I'm seeing, the second test should fail to allocate the string, but it fail before that, trying to allocate for a struct var. And so the logexpect hangs. passes: https://app.circleci.com/pipelines/github/varnish/varnish-modules/1391/workflows/e8315b8a-523f-4802-aec0-7207f4133bf5/jobs/2468 fails: https://app.circleci.com/pipelines/github/varnish/varnish-modules/1390/workflows/ed4ef4a0-f782-498f-a656-344179b15440/jobs/2467

I'm a bit perplexed as the commit doesn't seem to be related, so I thought I'd ask before diving in too deep