ViViDboarder / vaultwarden_ldap

Automate LDAP invites to Vaultwarden
GNU General Public License v3.0
157 stars 26 forks source link

Sync fails with VW 1.31 #145

Closed logopk closed 2 months ago

logopk commented 2 months ago

When syncing with an updated vaultwarden server with docker image 1.31.0 I get the following error:

ldap_sync-1    | Error inviting users from ldap. Count 4: Failed to get existing users from server
ldap_sync-1    | 
ldap_sync-1    | Caused by:
ldap_sync-1    |     0: Could not get list of existing users from server
ldap_sync-1    |     1: http error making request reqwest::Error { kind: Decode, source: Error("missing field `Email`", line: 1, column: 4315) }
ldap_sync-1    |     2: error decoding response body: missing field `Email` at line 1 column 4315
ldap_sync-1    |     3: missing field `Email` at line 1 column 4315

Reverting Vaultwarden to 1.30.5 is running without problem:

ldap_sync-1    | Existing user or invite found with email: xx@domain.com
ldap_sync-1    | Existing user or invite found with email: yy@domain.com
ldap_sync-1    | Existing user or invite found with email: zz@domain2.com
ldap_sync-1    | User with email already exists: xx@domain.com
ldap_sync-1    | User with email already exists: zz@domain2.com
ldap_sync-1    | Sent invites to 0 user(s).

Could it be that there is an issue with the change to camelCase in API and structs? https://github.com/dani-garcia/vaultwarden/pull/4386

Thanks.

Peter