Skip to content

Conversation

@asbiin
Copy link
Owner

@asbiin asbiin commented Mar 8, 2025

As explain on

The [default] timeout is set to null. The values recommended by the specification are as follow:

If the user verification is discouraged, timeout should be between 30 and 180 seconds

If the user verification is preferred or required, the range is 300 to 600 seconds (5 to 10 minutes)

These behaviors are not necessarily followed by the web browsers.

The Laravel cache put operation now converts the timeout from milliseconds to seconds.
If the timeout is null, then the maximum default recommendation is used for the cache operation.

Fix #496

@sonarqubecloud
Copy link

@asbiin asbiin merged commit 9b68ae9 into main Mar 11, 2025
21 checks passed
@asbiin asbiin deleted the timeout branch March 11, 2025 10:29
@github-actions
Copy link

🎉 This PR is included in version 5.0.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

The timeout configuration is declared as a millisecond unit but used as a second unit when doing cache operations

2 participants