[ 
https://issues.apache.org/jira/browse/GERONIMO-4847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Quintin Beukes updated GERONIMO-4847:
-------------------------------------

    Description: 
Adding a repository resource which already exists gives an HTTP 500 (from 
exception) error. It should instead prompt to override or show a friendlier 
message.

An example stack trace is:
java.lang.IllegalArgumentException: Destination 
/opt/kms/server/geronimo-2.1.4/repository/kms/GeronimoLoginModule/1.0/GeronimoLoginModule-1.0.jar
 already exists!
        
org.apache.geronimo.kernel.repository.AbstractRepository.copyToRepository(AbstractRepository.java:213)
        
org.apache.geronimo.kernel.repository.AbstractRepository.copyToRepository(AbstractRepository.java:189)
        
org.apache.geronimo.console.repository.RepositoryViewPortlet.processAction(RepositoryViewPortlet.java:169)
        org.apache.pluto.core.PortletServlet.dispatch(PortletServlet.java:218)
        org.apache.pluto.core.PortletServlet.doPost(PortletServlet.java:145)
        javax.servlet.http.HttpServlet.service(HttpServlet.java:713)
        javax.servlet.http.HttpServlet.service(HttpServlet.java:806)
        
org.apache.pluto.core.DefaultPortletInvokerService.invoke(DefaultPortletInvokerService.java:167)
        
org.apache.pluto.core.DefaultPortletInvokerService.action(DefaultPortletInvokerService.java:85)
        
org.apache.pluto.core.PortletContainerImpl.doAction(PortletContainerImpl.java:219)
        
org.apache.pluto.driver.PortalDriverServlet.doGet(PortalDriverServlet.java:121)
        
org.apache.pluto.driver.PortalDriverServlet.doPost(PortalDriverServlet.java:167)
        javax.servlet.http.HttpServlet.service(HttpServlet.java:713)
        javax.servlet.http.HttpServlet.service(HttpServlet.java:806)
        
org.apache.geronimo.console.filter.XSSXSRFFilter.doFilter(XSSXSRFFilter.java:125)


  was:
Adding a resource which already exists gives an HTTP 500 (from exception) 
error. It should instead prompt to override or show a friendlier message.

An example stack trace is:
java.lang.IllegalArgumentException: Destination 
/opt/kms/server/geronimo-2.1.4/repository/kms/GeronimoLoginModule/1.0/GeronimoLoginModule-1.0.jar
 already exists!
        
org.apache.geronimo.kernel.repository.AbstractRepository.copyToRepository(AbstractRepository.java:213)
        
org.apache.geronimo.kernel.repository.AbstractRepository.copyToRepository(AbstractRepository.java:189)
        
org.apache.geronimo.console.repository.RepositoryViewPortlet.processAction(RepositoryViewPortlet.java:169)
        org.apache.pluto.core.PortletServlet.dispatch(PortletServlet.java:218)
        org.apache.pluto.core.PortletServlet.doPost(PortletServlet.java:145)
        javax.servlet.http.HttpServlet.service(HttpServlet.java:713)
        javax.servlet.http.HttpServlet.service(HttpServlet.java:806)
        
org.apache.pluto.core.DefaultPortletInvokerService.invoke(DefaultPortletInvokerService.java:167)
        
org.apache.pluto.core.DefaultPortletInvokerService.action(DefaultPortletInvokerService.java:85)
        
org.apache.pluto.core.PortletContainerImpl.doAction(PortletContainerImpl.java:219)
        
org.apache.pluto.driver.PortalDriverServlet.doGet(PortalDriverServlet.java:121)
        
org.apache.pluto.driver.PortalDriverServlet.doPost(PortalDriverServlet.java:167)
        javax.servlet.http.HttpServlet.service(HttpServlet.java:713)
        javax.servlet.http.HttpServlet.service(HttpServlet.java:806)
        
org.apache.geronimo.console.filter.XSSXSRFFilter.doFilter(XSSXSRFFilter.java:125)


        Summary: HTTP 500 error from exception when adding a repository 
resource which already exists  (was: HTTP 500 error from exception when adding 
a resource which already exists)

> HTTP 500 error from exception when adding a repository resource which already 
> exists
> ------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4847
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4847
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: console
>    Affects Versions: 2.1.4
>            Reporter: Quintin Beukes
>
> Adding a repository resource which already exists gives an HTTP 500 (from 
> exception) error. It should instead prompt to override or show a friendlier 
> message.
> An example stack trace is:
> java.lang.IllegalArgumentException: Destination 
> /opt/kms/server/geronimo-2.1.4/repository/kms/GeronimoLoginModule/1.0/GeronimoLoginModule-1.0.jar
>  already exists!
>       
> org.apache.geronimo.kernel.repository.AbstractRepository.copyToRepository(AbstractRepository.java:213)
>       
> org.apache.geronimo.kernel.repository.AbstractRepository.copyToRepository(AbstractRepository.java:189)
>       
> org.apache.geronimo.console.repository.RepositoryViewPortlet.processAction(RepositoryViewPortlet.java:169)
>       org.apache.pluto.core.PortletServlet.dispatch(PortletServlet.java:218)
>       org.apache.pluto.core.PortletServlet.doPost(PortletServlet.java:145)
>       javax.servlet.http.HttpServlet.service(HttpServlet.java:713)
>       javax.servlet.http.HttpServlet.service(HttpServlet.java:806)
>       
> org.apache.pluto.core.DefaultPortletInvokerService.invoke(DefaultPortletInvokerService.java:167)
>       
> org.apache.pluto.core.DefaultPortletInvokerService.action(DefaultPortletInvokerService.java:85)
>       
> org.apache.pluto.core.PortletContainerImpl.doAction(PortletContainerImpl.java:219)
>       
> org.apache.pluto.driver.PortalDriverServlet.doGet(PortalDriverServlet.java:121)
>       
> org.apache.pluto.driver.PortalDriverServlet.doPost(PortalDriverServlet.java:167)
>       javax.servlet.http.HttpServlet.service(HttpServlet.java:713)
>       javax.servlet.http.HttpServlet.service(HttpServlet.java:806)
>       
> org.apache.geronimo.console.filter.XSSXSRFFilter.doFilter(XSSXSRFFilter.java:125)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to