benadida / helios-server

Helios server
http://heliosvoting.org
Apache License 2.0
711 stars 341 forks source link

Fix bug when removing password voter #377

Open Cryptosaurus opened 1 year ago

Cryptosaurus commented 1 year ago

voter.user is virtual for users of type password

Cryptosaurus commented 1 year ago

Proposed fix for #379

shirlei commented 1 year ago

@Cryptosaurus @benadida this seemed familiar to me and checking my fork I see that I ran into the same issue a while back and this is exactly what I did. My guess is that this use case isn't used a lot, but I think this PR would be nice to be accepted.

Cryptosaurus commented 1 year ago

I don't know how common this use-case is, but for my usage it would be a nice feature, and there is clearly an intention to support it in the current code.