All dependencies with these specific versions are already being used in our codebase.

This will be required for native. On Android we have to use rustls-tls because the native-tls bundles openssl which conflicts with our openssl setup during linking.

I added it in this diff to make sure that the commtest works with default features (native-tls).

Instead of passing url to each function call I added a struct that will hold the configuration.

Previously the user of the backup client passed in the blob hash. Now we just calculate it here.
If later we will need to be able to specify the hash value again (e.g. streaming the data?) we can just add e.g. user_data_hash: Option<String> to BackupData struct and do the hashing conditionally. But I prefer to keep it simple for now.

Previously it was:

Part::stream(Body::wrap_stream(
        stream! { yield Ok::<Vec<u8>, Infallible>(user_keys);  },
      )),

I changed it because I don't think it gave us anything -> we were still sending one big chunk (it wasn't really "streaming").

Previously this was converted to commtest-specific error. Now we just return reqwest::Error if something went wrong.

Previously this took ownership to BackupDescriptor. I changed it to take a reference as we don't need the complete ownership of the value inside the function.

Added an error struct.

To be honest I think derive_more get's us to about 95% percent of what thiserror gives us, with a slightly worse syntax. I would prefer to switch to thiserror but it would be better to change it throughout the whole codebase.