search

TouK / excel-export

excel-export grails plugin
58 stars 32 forks source link

UrlMappings should not be packaged with plugin #44

Closed houbie closed 7 years ago

houbie commented 8 years ago

The UrlMappings in the default package should be excluded from the plugin jar. It overwrites url mappings of the application. See also https://github.com/grails/grails-core/issues/9706

graemerocher commented 8 years ago

since they are in a package no I am not sure it is necessary from excluding them from being packaged, although we should update the documentation if this is the way we go. However plugin authors should remove the file if they don't plan to expose URL mappings.

dvisco commented 8 years ago

Is there any way to disable these mappings? as it stands the mappings are

        "/$controller/$action?/$id?"{
            constraints {
                // apply constraints here
            }
        }

        "/"(view:"/index")
        "500"(view:'/error')

If you utilize this plugin this mapping expose all controller actions which is problematic. Is there a way to ignore this mapping?