web/flows: fix authenticator_validate device select not sent to backend

[tograss]

Details

Resolves #5020
The old code version always returned without doing anything. I assume the intention was to ensure "value" is not undefined and is not false.

I tried the new version with sms and totp device. Having both active at the same time or only one. I did however not test a Yubikey /Webauthn.

Checklist

• Local tests pass (ak test authentik/)

• The code has been formatted (make lint-fix)
  If changes to the frontend have been made

• The code has been formatted (make web)

• The translation files have been updated (make i18n-extract) <- This creates a lot of changes not caused by my fix so I did not commit this .

[netlify]

✅ Deploy Preview for authentik ready!

Name	Link
🔨 Latest commit	3c7900e
🔍 Latest deploy log	https://app.netlify.com/sites/authentik/deploys/6463df86a6ea3c000831c5c2
😎 Deploy Preview	https://deploy-preview-5638--authentik.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: -0.02 ⚠️

Comparison is base (5213211) 92.63% compared to head (3c7900e) 92.61%.

❗ Current head 3c7900e differs from pull request most recent head 8398876. Consider uploading reports for the commit 8398876 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5638      +/-   ##
==========================================
- Coverage   92.63%   92.61%   -0.02%     
==========================================
  Files         546      546              
  Lines       26184    26184              
==========================================
- Hits        24254    24248       -6     
- Misses       1930     1936       +6     

Flag	Coverage Δ
e2e	52.00% <ø> (+0.10%)	⬆️
integration	26.39% <ø> (ø)
unit	89.50% <ø> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 5 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[BeryJu]

@tograss I think the error came from the fact that value was assigned to _selectedDeviceChallenge and then we compare value to _selectedDeviceChallenge and return if they are the same, which they always will be. I fixed that comparison and with that the correct request is sent to the backend when selecting a challenge

[tograss]

Ah now I see what is intended at this point.