Closed bilelmoussaoui closed 1 year ago
After a quick glance, looks good to me. Only one thing is a bit strange. In a point you used the base64 converter STANDARD and somewhere else STANDARD_NO_PAD.
You sure it's correct having different encodings? I don't know...
EDIT: ok, it was already like that before, so it's right
yes, it also has tests to ensure it is encoded/decoded correctly
The remaining ones are somehow related to actix & futures usage