Skip to content

Comments

Fix undefined form data usage in Confirm2Fa#6

Open
cannycookie wants to merge 1 commit into4.xfrom
lee/fix-undefined-variable-formdata-in-confirm2fa-xk0noh
Open

Fix undefined form data usage in Confirm2Fa#6
cannycookie wants to merge 1 commit into4.xfrom
lee/fix-undefined-variable-formdata-in-confirm2fa-xk0noh

Conversation

@cannycookie
Copy link
Collaborator

Summary

  • validate form fields directly and remove unused form state
  • pass field names to FilamentTwoFactor so codes are read from the request

Testing

  • php -l src/Livewire/Confirm2Fa.php
  • composer install --no-interaction --prefer-dist (fails: CONNECT tunnel failed, response 403)
  • vendor/bin/pest (fails: No such file or directory)

https://chatgpt.com/codex/tasks/task_e_68a66c60cee483339a71d1a4d964c395

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant