Skip to:

  1. Skip to Jira Navigation
  2. Skip to Side Navigation
  3. Skip to Main Content

Regression - Request timeout error failed to throw

Description

Lucee doesnt throws error for request timeout, if the request reach the request timeout. The regression starts from 5.3.8.190-SNAPSHOT onwards.

<cfsetting requesttimeout="2"> <!--- <cfdump var="#getPageContext().getRequestTimeout()#"> ---> <cfset sleep("10000")> <cfdump var="#now()#">

https://trycf.com/gist/ff6454844a4ae968cbc591fe4422e1bd/lucee5?theme=monokai

Maybe this commit causes this issue
https://github.com/lucee/Lucee/commit/095052a44bc315b05adcdb822395420fb27e59d5

Environment

None

Activity

Show:

Brad Wood 28 February 2022 at 18:43

Thanks for addressing this!

Michael Offner 28 February 2022 at 14:43

Brad Wood 25 November 2021 at 01:55

Regardless of whether a thread is in native code or not, Lucee should still interrupt it. The only thing you want to prevent is actually stopping the thread.

Brad Wood 25 November 2021 at 01:10

What is this status of this? So far as I can tell, Lucee’s request timeout is basically broken now. For example, a thread sleep call is shown as “native code”, but it is totally interruptible. Why wouldn’t Lucee interrupts threads like that?

Furthermore, the commit above doesn’t; even reference a ticket number, so I can’t even tell why it was made, what unit tests were written for it, or who tested the change.

Fixed

Details

Assignee

Reporter

Priority

Labels

Fix versions

New Issue warning screen

Before you create a new Issue, please post to the mailing list first https://dev.lucee.org

Once the issue has been verified, one of the Lucee team will ask you to file an issue

Sprint

Affects versions

Created 13 October 2021 at 15:27
Updated 28 February 2022 at 18:43
Resolved 28 February 2022 at 14:47

Flag notifications