Skip to content

Add on_send_message method to Handler#194

Open
dbrgn wants to merge 1 commit intohousleyjk:masterfrom
dbrgn:on_send_message
Open

Add on_send_message method to Handler#194
dbrgn wants to merge 1 commit intohousleyjk:masterfrom
dbrgn:on_send_message

Conversation

@dbrgn
Copy link

@dbrgn dbrgn commented Jan 8, 2018

This is analogue to on_send_frame. While on_send_frame is also called for messages prior to fragmentation, that's a low-level method called for any frame, no matter whether it's a user defined message or something else (like a frame from a close message).

This is analogue to `on_send_frame`. While `on_send_frame` is also
called for messages prior to fragmentation, that's a low-level method
called for any frame, no matter whether it's a user defined message or
something else (like a frame from a close message).
@housleyjk
Copy link
Owner

I am considering this, I don't know if I'm sold on it yet. It would be great to have more opinions, but I can see how it might be useful. I just don't have the time right now for a full review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants