Skip to content

Fix numeric select values for SHA512 TOTP - #1564

Open
cyphercodes wants to merge 1 commit into
Authenticator-Extension:devfrom
cyphercodes:fix-sha512-select-values
Open

Fix numeric select values for SHA512 TOTP#1564
cyphercodes wants to merge 1 commit into
Authenticator-Extension:devfrom
cyphercodes:fix-sha512-select-values

Conversation

@cyphercodes

Copy link
Copy Markdown

Summary

  • Preserve typed option values in SelectInput instead of coercing them through string DOM values.
  • Adds coverage for numeric select values so SHA512/other enum options are passed through correctly.

Fixes #1559

Testing

  • npm run test -- src/test/components/common/SelectInput.test.ts
  • npm run lint
  • ./node_modules/.bin/prettier --check src/components/common/SelectInput.vue src/test/components/common/SelectInput.test.ts
  • git diff --check

Note: full xvfb-run -a npm test was attempted locally but the extension browser harness was blocked by the environment (net::ERR_BLOCKED_BY_CLIENT).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SHA512 code incorrect

1 participant