Use prepend to patch Action Cable classes

[palkan]

To preserve the original functionality a better monkey-patching technique than copying the code should be used, e.g.,Module#prepend.

NOTE: The original Action Cable code for Connection class has been already changed, thus Lograge breaks the core functionality:

  rescue ActionCable::Connection::Authorization::UnauthorizedError
-  respond_to_invalid_request		
+  close(reason: ActionCable::INTERNAL[:disconnect_reasons][:unauthorized], reconnect: false) if websocket.alive?
  end

Fixes #304.
Related to #294.

[danielnc]

bump! we are seeing this issue too

[cin210]

We're able to reproduce this as well. Good fix!

[palkan]

Yes, this issue still makes Lograge incompatible with AnyCable; we're currently using this patch.

[jefflub]

Bump! Now using a patched version of lograge to work around this. Thanks @palkan!

[iloveitaly]

@palkan could you rebase this on master so CI can run? Can you add an entry to the changelog for this too?

Would love to get this merged in.

[daande]

We are seeing builds failing now since we pinned to commit 41a772a31ce39c6989d6d6e8c736e72247f2b620 I guess we can use the branch name instead. Would like to see this merged in asap please 💯

[palkan]

We are seeing builds failing now

Oops, sorry 🙄 (Highly recommend to fork all source-based dependencies to avoid such problems in the future)

[iloveitaly]

@palkan Awesome change! Mind submitting another PR with a changelog entry?

Hoping to cut a new release soon.