Aurélien Lajoie comments

Results 23 comments of


                                            Aurélien Lajoie

feat(proxy): Allow to remove the proxy cache headers

@hbagdi I have described the impact. > Remove proxy cache headers from response if the kong debug is not set. Nothing has changed on the caching part, just the headers...

feat(proxy): Allow to remove the proxy cache headers

Not sure that not being able to not sent the header was acceptable. If you think this is a breaking change, I can add a flag to add them or...

feat(proxy): Allow to remove the proxy cache headers

I was planning to add a flag for each header, to put it on the answer. set to true by default to keep the behavior And force all on debug...

feat(proxy): Allow to remove the proxy cache headers

I have added a response_headers field into the configuration with 3 fields: * age * X-Cache-Status * X-Cache-Key Allowing to set or unset the presence of the header into the...

feat(proxy): Allow to remove the proxy cache headers

Not sure what is missing, just rewords the Merge request? On Mon, Jun 26, 2023, 20:00 Harry ***@***.***> wrote: > Closed #10445 . > > — > Reply to this...

feat(proxy): Allow to remove the proxy cache headers

I have reword the PR, changelog commit is done on my fork, I don't have permission to reopen the PR. Not sure what is a `compat change`, here as asked...

feat(proxy): Allow to remove the proxy cache headers

@hanshuebner is it good now ? If something else is needed can you highlight it ?

feat(proxy): Allow to remove the proxy cache headers

Thanks for your support to make it happen

fix(proxy-cache) handle 204/304 responses without Content-Type

not interested today on my side

Diff coloring of each file is good but the diff coloring with 2 files seems broken

[diff.txt](https://github.com/so-fancy/diff-so-fancy/files/10764264/diff.txt)