You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ev...@apache.org on 2007/03/05 21:25:58 UTC

svn commit: r514852 - /maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigurationAction.java

Author: evenisse
Date: Mon Mar  5 12:25:57 2007
New Revision: 514852

URL: http://svn.apache.org/viewvc?view=rev&rev=514852
Log:
[CONTINUUM-1201] Fix Authorization error after installation

Modified:
    maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigurationAction.java

Modified: maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigurationAction.java
URL: http://svn.apache.org/viewvc/maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigurationAction.java?view=diff&rev=514852&r1=514851&r2=514852
==============================================================================
--- maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigurationAction.java (original)
+++ maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigurationAction.java Mon Mar  5 12:25:57 2007
@@ -90,6 +90,11 @@
             getLogger().info( "baseUrl='" + baseUrl + "'" );
         }
     }
+    
+    public String input()
+    {
+        return INPUT;
+    }
 
     public String save()
         throws ConfigurationStoringException, ContinuumStoreException
@@ -157,9 +162,11 @@
     public SecureActionBundle getSecureActionBundle()
         throws SecureActionException
     {
+        getLogger().info( "1");
         SecureActionBundle bundle = new SecureActionBundle();
         bundle.setRequiresAuthentication( true );
         bundle.addRequiredAuthorization( ContinuumRoleConstants.CONTINUUM_MANAGE_CONFIGURATION, Resource.GLOBAL );
+        getLogger().info( "2");
 
         return bundle;
      }

Re: svn commit: r514852 - /maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigurationAction.java

Posted by Emmanuel Venisse <em...@venisse.net>.
already fixed, but thanks :)

Brett Porter a écrit :
> 
> On 05/03/2007, at 12:25 PM, evenisse@apache.org wrote:
> 
>> +        getLogger().info( "1");
>>          SecureActionBundle bundle = new SecureActionBundle();
>>          bundle.setRequiresAuthentication( true );
>>          bundle.addRequiredAuthorization( 
>> ContinuumRoleConstants.CONTINUUM_MANAGE_CONFIGURATION, Resource.GLOBAL );
>> +        getLogger().info( "2");
> 
> oops?
> 
> 
> 
> 
>

Re: svn commit: r514852 - /maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigurationAction.java

Posted by Brett Porter <br...@apache.org>.
On 05/03/2007, at 12:25 PM, evenisse@apache.org wrote:

> +        getLogger().info( "1");
>          SecureActionBundle bundle = new SecureActionBundle();
>          bundle.setRequiresAuthentication( true );
>          bundle.addRequiredAuthorization 
> ( ContinuumRoleConstants.CONTINUUM_MANAGE_CONFIGURATION,  
> Resource.GLOBAL );
> +        getLogger().info( "2");

oops?