make access_all optional

[stefan0xC]

As noted here #4775 (comment) the access_all flag has been removed in the newer web-vaults resulting in an error when editing a user:

[2024-08-01 13:29:25.313][vaultwarden::api::core::organizations::_][WARN] Data guard `Json < EditUserData >` failed: Parse("{\"collections\":[{\"id\":\"64256828-6251-4bef-8dee-2128b63f520a\",\"readOnly\":true,\"hidePasswords\":false,\"manage\":false}],\"groups\":[],\"permissions\":{\"response\":null},\"type\":0}", Error("missing field `accessAll`", line: 1, column: 169)).

Note: Since newer web-vaults don't have the access_all flag anymore, it will be removed if you update a user with that permission. Due to the new view admins/owners will also not see all items in the password manager anymore, but only to items in collections they have explicit view access to (either directly or via a group).

This is purely a fix to make it compatible with newer versions of the web-vault (web-v2024.6.3 and later). Bitwarden stated that they will deprecate "the Access all existing and future collections permission" and change "all users that had this permission the Can manage permission for all existing collections" when migrating existing users to the new collection management system. This does not (currently) apply to Vaultwarden as we have not implemented this yet. They also removed the Manager role, so I'm not sure if that's the appropriate course of action, giving all users with access_all the ability to manage who has access to collections... 😬

[dani-garcia]

We should be able to tag the field with serde(default) to avoid needing to unwrap it in multiple places

Suggested change

	access_all: Option<bool>,
	#[serde(default)]
	access_all: bool,

[stefan0xC]

shall I change that in the InviteData and GroupRequest structs too?

[dani-garcia]

If it's not too much work sure, otherwise we can do those on a separate PR

[BlackDex]

Not sure from the top of my head, but it's the default defaulting to false?

[stefan0xC]

Yes. https://doc.rust-lang.org/std/primitive.bool.html#impl-Default-for-bool