Probably put unreachable!()
in else branch ? Because get_existing_bucket does not return a bucket that is deleted. (I think so, this should be checked)
Why break the convention that bucket name = domain name of the hosted website ? This gives more flexibility as we can host sites on diffferent tld/sld. If a website needs to move, we will copy files to the new bucket, it shouldn't be too costly (we don't expect to store TBs of data for a website...)
roxmltree, rustls, webpki, httpdate and maybe some others can probably be removed
NO!! expect
makes us crash (panic) if the condition is not verified, this is not acceptable (we don't want to assume that authority is well formed, maybe hyper checks it for us but maybe not and we don't want to take risks)