-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hex Viewer freezes SI #462
Conversation
@dannycohen have you heard much from our users whether the hex viewer is useful or not ? |
@johnsimons - hex viewer might make sense when the serialization is binary. That said I don't know how many people are using binary serialization and looking for a hex-view. Let me get back to you based on support cases / old history. |
It is also worth mentioning that at the moment our hex viewer does not On Thursday, May 21, 2015, Indu Alagarsamy notifications@github.com wrote:
|
Team, coincidently my manager wanted to user binary serialization initially because he had concerns about performance and the number of messages we were publishing. We did not implement the binary serialization, but if we did the hex viewer would be invaluable. As for the copy and paste feature, we noticed the lack of it as well. Of course we could always take the raw message body and throw it in our own utility or notepad++ to view the binary contents, but that just does not feel right for a product of this caliber. |
@distantcam here is what I did to address the issue with the hex viewer. |
@distantcam one more code review ? |
When a message is a bit bigger then normal, the UI in SI freezes and becomes unresponsive trying to render the hex viewer.
Example of a larger message body: