mirrors/cosmopolitan
1
0
Fork 0
mirror of https://github.com/jart/cosmopolitan.git synced 2025-01-31 11:37:35 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix redbean to avoid compressing the first yielded chunk. (#813)

Browse source 
This may happen if no Status or Content-Type is set and if the written
content is longer than 100B, the compression is going to be applied
after coroutine.yield, even though it shouldn't be.
This commit is contained in:
Paul Kulchenko 2023-05-17 20:46:27 -07:00 committed by GitHub
parent e7eb0b3070
commit 021c71b651
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
tool/net/redbean.c
View file

@ -2447,7 +2447,7 @@ static char *CommitOutput(char *p) {
size_t outbuflen;
if (!cpm.contentlength) {
outbuflen = appendz(cpm.outbuf).i;
if (cpm.istext && outbuflen >= 100) {
if (cpm.istext && !cpm.isyielding && outbuflen >= 100) {
if (!IsTiny() && !IsSslCompressed()) {
p = stpcpy(p, "Vary: Accept-Encoding\r\n");
}
@ -2610,7 +2610,6 @@ static int LuaCallWithYield(lua_State *L) {
YL = co;
cpm.generator = YieldGenerator;
if (!cpm.isyielding) cpm.isyielding = 1;
cpm.istext = false; // reset istext flag to avoid zipping yielded chunk
status = LUA_OK;
}
return status;
@ -6185,9 +6184,10 @@ static char *SetStatus(unsigned code, const char *reason) {
}
cpm.statuscode = code;
cpm.hascontenttype = false;
// reset, as the headers are reset
// istext is -1 by default to interpret as true when not set
cpm.istext = -1;
// reset, as the headers are reset.
// istext is `true`, as the default content type
// is text/html, which will be set later
cpm.istext = true;
cpm.gotxcontenttypeoptions = 0;
cpm.gotcachecontrol = 0;
cpm.referrerpolicy = 0;