chproxy icon indicating copy to clipboard operation
chproxy copied to clipboard

chore: Handle too big CH payloads for caching

Open sigua-cs opened this issue 3 years ago • 3 comments

Implements https://github.com/ContentSquare/chproxy/issues/143

sigua-cs avatar Jul 19 '22 12:07 sigua-cs

Your Render PR Server URL is https://chproxy-pr-191.onrender.com.

Follow its progress at https://dashboard.render.com/static/srv-cbba2pb0tnuvdk1hnv20.

render[bot] avatar Jul 19 '22 12:07 render[bot]

Your Render PR Server URL is https://chproxy-pr-191.onrender.com.

Follow its progress at https://dashboard.render.com/static/srv-cbbak6givq07b7e6n7d0.

render[bot] avatar Jul 19 '22 12:07 render[bot]

I need to update the proxy.go according the latest fix version

sigua-cs avatar Jul 20 '22 16:07 sigua-cs

I also don't think distinguishing http vs https is necessary in tests. It's enough to test one given we're interested in checking for if max payload size is exceeded or not @sigua-cs

gontarzpawel avatar Sep 09 '22 07:09 gontarzpawel

I also don't think distinguishing http vs https is necessary in tests. It's enough to test one given we're interested in checking for if max payload size is exceeded or not @sigua-cs

Removed HTTP tests. Only HTTPS will be used

sigua-cs avatar Sep 09 '22 11:09 sigua-cs