search

fippo / rtcstats

MIT License
129 stars 41 forks source link

add message compression #21

Open fippo opened 8 years ago

fippo commented 8 years ago

from #18: lets also compress this, either using lzstring or a using a custom name mapping. Most likely lzstring which adds ~4kb download is the best solution.

ggarber commented 8 years ago

1.6Kb gzipped https://cdnjs.cloudflare.com/ajax/libs/lz-string/1.4.4/lz-string.min.js

There is a websocket deflate extension. If we figure out how to enable it maybe we can get that with 0 effort and 0 extra KB and dependencies.

fippo commented 8 years ago 
Sec-WebSocket-Extensions:permessage-deflate

-- already on? I had wondered because the actual traffic measured in wireshark was way lower than I expected when I looked. How did you actually measure, looking at the frames or with wireshark?

fippo commented 8 years ago

i think we resolved this, did we?

ggarber commented 8 years ago

I thin we still didn't add lzstring, right? But feel free to close it and I will create another if needed later.

fippo commented 7 years ago

https://github.com/nodeca/pako -- we could use this