safe_devices does not remember device about twofactor

[michelterstege81]

wantsToAddDevice() returns false. I'm using the default view that comes with the package for the 2FA challenge.

wantsToAddDevice() checks for $this->request->filled('safe_device'), which is not in the request, since the view does not contain that field in the form.

Maybe this is just a documentation issue. I guess this only works when creating a custom view that uses a safe_device field.

[DarkGhostHunter]

Indeed, wantsToAddDevice() checks if the request contains safe_device and is not empty.

It could be added to the view if the safe_devices config is true, and hide it when it's disabled. Since I'm hands full right now, not on top of my list, but you're welcome to make a PR.

[Waleedviews]

@DarkGhostHunter i have exactly the same issue, how could i turn it on? i have added safe_device as a checkbox and is showing as on when i click on checkbox however it still shows safe_devices as a null, may be i am doing something wrong.

[DarkGhostHunter]

Edit the included view  file Italo Baeza C.El 07-03-2023, a la(s) 17:43, Waleed Afridi ***@***.***> escribió: @DarkGhostHunter i have exactly the same issue, how could i turn it on? —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>

[Waleedviews]

@DarkGhostHunter could you please elaborate this?

[DarkGhostHunter]

Publish the view and edit the form.

[Waleedviews]

i already publish the view called confirm.blade.php, i have edit the form

<input id="safe_device" type="checkbox" class="border border-gray-300 name="safe_device" />
<label for="safe_device"> I trust this device. Don't ask for codes for 14 days. </label>

but nothing happened, it doesn't store the safe_devices data.

[Waleedviews]

@DarkGhostHunter checking to see if you have any update on this? thank you.

[DarkGhostHunter]

@DarkGhostHunter checking to see if you have any update on this? thank you.

Hands full. PR welcomed.

[AlexisSerneels]

Here is the correct code you need to use:

<input id="safe_device" type="checkbox" value="1" class="border border-gray-300" name="safe_device" />
                <label for="safe_device"> I trust this device. Don't ask for codes for 14 days. </label>

You were missing the value and the class attribute was not closed.

[Waleedviews]

@AlexisSerneels i try this and it doesn't work, i am sure there's some issue with the code, i haven't check the package code yet, is it working with you?