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

ESI status check vs. vcl_synth status > 1000 #3958

Closed
nigoroll opened this issue Jul 12, 2023 · 0 comments
Closed

ESI status check vs. vcl_synth status > 1000 #3958

nigoroll opened this issue Jul 12, 2023 · 0 comments

Comments

@nigoroll
Copy link
Member

Expected Behavior

A synthetic status >1000 should behave like status % 1000

Current Behavior

A synthetic status >1000 triggers an ESI abort

Possible Solution

Fix coming up

Steps to Reproduce (for bugs)

No response

Context

Pro-Forma Ticket

Varnish Cache version

81a0a30

Operating system

No response

Source of binary packages used (if any)

No response
@nigoroll nigoroll mentioned this issue Jul 12, 2023
Merged
nigoroll added a commit to nigoroll/varnish-cache that referenced this issue Jul 12, 2023
@nigoroll
ESI: Use response status modulo 1000
9cc681c 
Ref: 582ded6

Fixes varnishcache#3958
Dridi pushed a commit to Dridi/varnish-cache that referenced this issue Jul 13, 2023
@nigoroll @Dridi
ESI: Use response status modulo 1000
5636ad0 
Ref: 582ded6

Fixes varnishcache#3958
Dridi pushed a commit that referenced this issue Oct 25, 2023
@nigoroll @Dridi
ESI: Use response status modulo 1000
aacf777 
Ref: 582ded6

Fixes #3958
daghf pushed a commit to daghf/varnish-cache that referenced this issue Nov 7, 2023
@nigoroll @daghf
ESI: Use response status modulo 1000
aac0a05 
Ref: 582ded6

Fixes varnishcache#3958
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nigoroll