<div dir="ltr"><div><div>Hi,<br><br>I had some time in my recent flights so I decided to give it a shot.<br></div>As discussed at VDD15Q1, this changes the hash_data input to BLOB and adds automatic conversion from STRING to BLOB.<br></div><div><br></div><div>There are a few downsides with this approach:<br><br>1. Every time we call hash_data() on something else than a BLOB we'll perform the conversion. <br>If we assume that most people will not hash the body (a fair assumption I believe) we're doing extra work that only favours a minority, penalising the rest.<br><br></div><div>2. Adds more pressure to the already tight workspace. I guess we could use something else here if required.<br><br></div><div>The other thing that bothers me somewhat is the naming behind the BLOBs, vmod_priv.<br></div><div>If BLOBs will become a first class citizen we should rename it to something that is not coupled to vmods. This is regardless of the direction we take.<br><br></div><div>A better alternative would be to teach vcc to issue two different calls depending on the input, e.g. for BLOBs it will use VRT_hashblob(), for anything else use VRT_hashstring(), so point 1 and 2 only affect users of hash_data(req.body).<br></div><div><br></div><div>Comments? OKs?<br></div><div><br></div></div>