Redirect to an non existing action sometimes hangs

Details

Type: Bug
Status: Open
Priority: Major
Resolution: Unresolved
Affects Version/s: 2.0-RC3
Fix Version/s: None
Component/s: Controllers
Labels:

Description

The behavior of redirect to an non existing action is inconsistent. See example below.

TestController.groovy

package foo
class TestController {
	
	def foo() {
		redirect(action: noaction)      // hangs
		//redirect(action: "noaction")  // returns 404 error code, correct semantically
		//redirect(action:'bar')        // works correctly
		//redirect(action:bar)		// hangs, notice no quotes aroud bar
	}
	
	def bar() {
		render "bar action"
	}
}

It would be nice if instead of hanging it explicitly failed with an exception.

Activity

Ascending order - Click to sort in descending order

numan salati made changes - 07/Dec/11 11:53 AM

Field	Original Value	New Value
Status	Open [ 1 ]	Edit [ 10000 ]
Description	The behavior of redirect to an non existing action is inconsistent. See example below. package foo class TestController { def foo() { redirect(action: noaction) // hangs //redirect(action: "noaction") // returns 404 error code //redirect(action:'bar') // works correctly //redirect(action:bar) // hangs } def bar() { render "bar action" } } It would be nice if instead of hanging it explicitly failed with an exception.	The behavior of redirect to an non existing action is inconsistent. See example below. {code:title=TestController.groovy\|borderStyle=solid} package foo class TestController { def foo() { redirect(action: noaction) // hangs //redirect(action: "noaction") // returns 404 error code //redirect(action:'bar') // works correctly //redirect(action:bar) // hangs } def bar() { render "bar action" } } {code} It would be nice if instead of hanging it explicitly failed with an exception.

numan salati made changes - 07/Dec/11 11:54 AM

Status

Edit [ 10000 ]

Open [ 1 ]

numan salati made changes - 07/Dec/11 11:55 AM

Status	Open [ 1 ]	Edit [ 10000 ]
Description	The behavior of redirect to an non existing action is inconsistent. See example below. {code:title=TestController.groovy\|borderStyle=solid} package foo class TestController { def foo() { redirect(action: noaction) // hangs //redirect(action: "noaction") // returns 404 error code //redirect(action:'bar') // works correctly //redirect(action:bar) // hangs } def bar() { render "bar action" } } {code} It would be nice if instead of hanging it explicitly failed with an exception.	The behavior of redirect to an non existing action is inconsistent. See example below. {code:title=TestController.groovy\|borderStyle=solid} package foo class TestController { def foo() { redirect(action: noaction) // hangs //redirect(action: "noaction") // returns 404 error code //redirect(action:'bar') // works correctly //redirect(action:bar) // hangs, notice no quotes aroud bar } def bar() { render "bar action" } } {code} It would be nice if instead of hanging it explicitly failed with an exception.

numan salati made changes - 07/Dec/11 11:55 AM

Status

Edit [ 10000 ]

Open [ 1 ]

numan salati made changes - 07/Dec/11 11:56 AM

Status	Open [ 1 ]	Edit [ 10000 ]
Description	The behavior of redirect to an non existing action is inconsistent. See example below. {code:title=TestController.groovy\|borderStyle=solid} package foo class TestController { def foo() { redirect(action: noaction) // hangs //redirect(action: "noaction") // returns 404 error code //redirect(action:'bar') // works correctly //redirect(action:bar) // hangs, notice no quotes aroud bar } def bar() { render "bar action" } } {code} It would be nice if instead of hanging it explicitly failed with an exception.	The behavior of redirect to an non existing action is inconsistent. See example below. {code:title=TestController.groovy\|borderStyle=solid} package foo class TestController { def foo() { redirect(action: noaction) // hangs //redirect(action: "noaction") // returns 404 error code, correct semantically //redirect(action:'bar') // works correctly //redirect(action:bar) // hangs, notice no quotes aroud bar } def bar() { render "bar action" } } {code} It would be nice if instead of hanging it explicitly failed with an exception.