stimulus_reflex icon indicating copy to clipboard operation
stimulus_reflex copied to clipboard

permessage deflate

Open leastbad opened this issue 5 years ago • 0 comments

Type of PR (feature, enhancement, bug fix, etc.)

Enhancement

Description

My [currently failed] attempt to enable permessage_deflate as a monkey patch in SR.

Before: chrome_Dx9qik8Txm After: chrome_DsMCPolonR

Despite promising initial results, it's shitting the bed with segfaults and I don't really know what seems to be the problem. Seeing "Invalid frame header" responses. What's weird is that some messages seem to manage to get through.

I've tried all of the possible values of :max_window_bits from 8 to 15.

My current solution is stolen verbatim from https://sapandiwakar.in/compressing-websocket-actioncable-frames-on-rails-with-permessage-deflate/ and was helped/influenced by https://github.com/faye/permessage-deflate-ruby

Anyhow, just sharing here in case someone has some detective juice.

Why should this be added

Trying to enable brotli compression for WS traffic.

Checklist

  • [x] My code follows the style guidelines of this project
  • [x] Checks (StandardRB & Prettier-Standard) are passing
  • [x] This is not a documentation update

leastbad avatar Feb 08 '21 08:02 leastbad