CyberChef icon indicating copy to clipboard operation
CyberChef copied to clipboard

Add option to parse non-varint Protobuf fields as floats

Open chrnola opened this issue 4 years ago • 2 comments

Protobuf wire types 1 and 5 represent fixed-sized 64-bit and 32-bit values, respectively (see docs). However the representation on the wire contains no way to differentiate encoded integers from floating point values.

The current implementation can be confusing to users who are attempting to decode a Protobuf messages that contains float or double fields since they will always be decoded as integers.

This PR introduces a new argument to the Protobuf Decode operation that allows users to opt-in to parsing fields of wire types 1 and 5 as floats.

chrnola avatar Sep 14 '20 17:09 chrnola

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Sep 14 '20 17:09 CLAassistant

Hey! This pull request has gotten rather old and that's our fault. This project stopped being actively maintained for a while and it looks like your pull request has started to gather dust. Would you be able to update your branch to the latest version of CyberChef and we'll give it a review?

a3957273 avatar Apr 02 '24 20:04 a3957273

Hey, this PR has remained without updates for four weeks now. I'll close it now, but feel free to re-open if you find time to contribute! 😄

a3957273 avatar May 05 '24 21:05 a3957273