vaultwarden icon indicating copy to clipboard operation
vaultwarden copied to clipboard

Change API and structs to camelCase

Open dani-garcia opened this issue 4 months ago • 2 comments

This PR changes all the project's structs and API inputs and outputs from the old PascalCase format to the new camelCase format. At the moment the clients support both but that won't be a thing forever.

I haven't had the time to fully test this, but at least the basic things work. There may be some things missing, and I still need to review what we store in the db to make sure that there aren't any backwards compat issues.

This might also cause problems with other pending PRs so we can wait to merge it until they are dealt with.

Some fields that have special casing:

  • OTP in /accounts/verify-otp, as that's what the clients send. That said we also check otp
  • Keys and Nfc in yubikey, this is for backwards compat with older values stored on the server, now we serialize new versions to keys and nfc.
  • AttestationObject and clientDataJson in webauthn, as that's what the clients send. This is a bigger annoyance as it complicates the use of the webauthn crate.
  • Organization policies, this is for backwards compat with older values stored on the server, now we serialize new versions as camelCase.

dani-garcia avatar Feb 28 '24 00:02 dani-garcia