[GROOVY-5935] GroovyServlet script resolving broken

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 1.8.8, 2.0.5, 2.3.0
• Component/s: Groovlet / GSP
• Assignee: Unassigned
• Reporter: Wolfgang Schell
• Created: 21/Jan/13 03:20
• Updated: 13/Apr/15 08:57

Description

When using groovy script with the GroovyServlet, script resolving is currently broken. It works the first time, but as soon as the entry is cached, resolving no longer works. groovy.servlet.AbstractHttpServlet tries to resolve the script file in order to determine the last modification time, but fails to do this with file: URLs.

This commit breaks the resource loading in groovy.servlet.AbstractHttpServlet:
https://github.com/groovy/groovy-core/commit/e941c940655bbdf0b62c60ff02754e993394b3a8#src/main/groovy/util/GroovyScriptEngine.java

The commit above changed the path being used for resource loading to a full URL with a file: scheme, which is not taken into account when loading resources in groovy.servlet.AbstractHttpServlet.getResourceConnection(String)

An easy fix would be to change groovy.servlet.AbstractHttpServlet to parse the provided name as URI and get the path from it.