Re: applications re-org

[Jaq Marit]

Quoting Roger Ruttimann <[EMAIL PROTECTED]>:

> +1 for the reorg of applications
> 
> Why not just using portlets instead of general. In a subfolder it could 
> be grouped by functionality such as webcontent (iframe,..), utilities 
> (localselector, calendar,..).
> 

I think grouping by functionality is better rather than tagging it as 'general'
because some portlets may not be considered generally useful to all
orgranizations (e.g., stock quote portlet, calendar, etc... may not be present
in all portals).  Just a thought...


This message was sent using IMP, the Internet Messaging Program.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: portal project deployment in WEB-INF/classes vs. WEB-INF/lib

[David Sean Taylor]

Randy Watler wrote:
David,
I think this is (or was) Maven's standard way of packaging a war file.
Im not opposed to changing it, but if you do, please make sure nothing 
else breaks (such as the hotdeploy goal)

Really? Wow. I have never used WEB-INF/classes for anything other than 
properties files. Ok, I'll look into it more carefully.

try this:
mkdir someproject
cd someproject
maven genapp  (use the default template)
maven war
With the default template, it seems to put your classes in the classes 
directory.

The servlet spec clearly allows you to put classes there, its a matter 
of best practices and preferences.

--
David Sean Taylor
Bluesunrise Software
[EMAIL PROTECTED]
[office] +01 707 773 4646
[mobile] +01 707 529 9194
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: applications re-org

[Shinsuke SUGAYA]

+1
David Sean Taylor wrote:
in preparation for m1, propose to reorg the applications folder.
currently we have:
+ demo
+ jsf-demo
+ localeselector
+ pam
+ perl
+ php
+ security
+ struts-demo
into
+ demo
+ pam
+ security
+ general (new)
+ rss (xml, syndication) (new)
where:
php, perl, jsf-demo, struts-demo move to -> demo
localeselector, (and iframeportlet) move to -> general
not sure if 'general' if the best name
im thinking of putting general (theres that word again) useful nondemo 
portlets such as localeselector, iframeportlet, calendar (coming soon), 
bookmark, weather, stockquote etc since they are not really demo so much 
as real useful portlets.

for the rss pa, i plan to port the J1 rss, xslt and bbc portlets
other names for this pa could be xml or syndication
suggestions objections welcome
__
Do You Yahoo!?
Upgrade Your Life
http://bb.yahoo.co.jp/
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: portal project deployment in WEB-INF/classes vs. WEB-INF/lib

[Randy Watler]

David,
I think this is (or was) Maven's standard way of packaging a war file.
Im not opposed to changing it, but if you do, please make sure nothing 
else breaks (such as the hotdeploy goal)
Really? Wow. I have never used WEB-INF/classes for anything other than 
properties files. Ok, I'll look into it more carefully.

Looking at RdbmsPolicy.getDefaultPolicy():
I
private static String defaultPolicy = "sun.security.provider.PolicyFile";
then
  Class policyClass = Class.forName(RdbmsPolicy.defaultPolicy);
return (Policy) policyClass.newInstance();
Is that where the performance hit is?
Or is it in getPermissons(). This code also looks suspect to me, 
especially the policy.refresh():

// TODO Is there a better way to do this?
// If the permission is not found here then delegate it
// to the standard java Policy class instance.
Policy.setPolicy(RdbmsPolicy.getDefaultPolicy());
Policy policy = Policy.getPolicy();
policy.refresh();
PermissionCollection defaultPerms = 
policy.getPermissions(codeSource);
// Revert back to the current policy.
Policy.setPolicy(this);
// Return the default permission collection.
return defaultPerms;

The issue is that RdbmsPolicy.getPermissions() gets called many times 
for a single checkPermission() invocation, (once per 
CodeSource/PermissionDomain). In the AbstractPageManager case, it is 
called 4-6 instead of perhaps 2-3 times per unique checkPermission() per 
user session. Each time, it invokes PermissionManager.getPermissions() 
which hits the DB, (or look like it does anyway). Worse case, we might 
consider caching in the principal impls to eliminate redundant lookup. 
Ideally, I'd like to get it down to just one access 
RdbmsPolicy.getPermissions(), but I do not know how to use the 
Subject.doAs*() methods to accomplish that yet. DLS?

Randy

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: applications re-org

[Ate Douma]

+1

David Sean Taylor said:
> in preparation for m1, propose to reorg the applications folder.
> currently we have:
>
> + demo
> + jsf-demo
> + localeselector
> + pam
> + perl
> + php
> + security
> + struts-demo
>
> into
>
> + demo
> + pam
> + security
> + general (new)
> + rss (xml, syndication) (new)
>
> where:
>
> php, perl, jsf-demo, struts-demo move to -> demo
> localeselector, (and iframeportlet) move to -> general
>
> not sure if 'general' if the best name
> im thinking of putting general (theres that word again) useful nondemo
> portlets such as localeselector, iframeportlet, calendar (coming soon),
> bookmark, weather, stockquote etc since they are not really demo so much
> as real useful portlets.
>
> for the rss pa, i plan to port the J1 rss, xslt and bbc portlets
> other names for this pa could be xml or syndication
>
> suggestions objections welcome
>
> --
> David Sean Taylor
> Bluesunrise Software
> [EMAIL PROTECTED]
> [office] +01 707 773 4646
> [mobile] +01 707 529 9194
>
>
> -
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

RE: security valve

[Ate Douma]

Randy Watler said:
> David,
>
> ...or, you can just wait for my next patch and it will go in, (assuming
> Ate
> approves).
It looks good to me too, but as I haven't been able to test it yet, the
only reservation I have is (as I already said before) that the Login
Portlet still works as expected.
If it does, I'm +1.

Ate
>
> Thanks,
>
> Randy
>
> -Original Message-
> From: David Sean Taylor
> To: Jetspeed Developers List
> Sent: 11/11/04 11:44 PM
> Subject: Re: security valve
>
> Randy Watler wrote:
>
>> Ate/David:
>>
>> I am now using the code below successfully in SecurityValveImpl.java.
>>
>> Randy
>>
>
> Looks good to me.
> With Ate's blessing I'll commit it
>
>


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

cvs commit: jakarta-jetspeed-2/taglibs/treecontrol/src/java/org/apache/webapp/admin TreeControl.java

[taylor]

taylor  2004/11/12 11:36:08

  Modified:taglibs/treecontrol/src/java/org/apache/webapp/admin
TreeControl.java
  Log:
  put unique check back in
  
  Revision  ChangesPath
  1.3   +4 -4  
jakarta-jetspeed-2/taglibs/treecontrol/src/java/org/apache/webapp/admin/TreeControl.java
  
  Index: TreeControl.java
  ===
  RCS file: 
/home/cvs/jakarta-jetspeed-2/taglibs/treecontrol/src/java/org/apache/webapp/admin/TreeControl.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- TreeControl.java  11 Nov 2004 19:22:56 -  1.2
  +++ TreeControl.java  12 Nov 2004 19:36:08 -  1.3
  @@ -167,9 +167,9 @@
   
   synchronized (registry) {
   String name = node.getName();
  -//if (registry.containsKey(name))
  -//throw new IllegalArgumentException("Name '" + name +
  -//   "' is not unique");
  +if (registry.containsKey(name))
  +throw new IllegalArgumentException("Name '" + name +
  +   "' is not unique");
   node.setTree(this);
   registry.put(name, node);
   }
  
  
  

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

cvs commit: jakarta-jetspeed-2/portals-bridges/frameworks/src/java/org/apache/portals/bridges/frameworks Lookup.java GenericFrameworkPortlet.java

[taylor]

taylor  2004/11/12 11:35:46

  Modified:
portals-bridges/frameworks/src/java/org/apache/portals/bridges/frameworks
Lookup.java GenericFrameworkPortlet.java
  Log:
  - add bean parameter to actions
  - support for isNew for typical form beans
  
  Revision  ChangesPath
  1.2   +2 -1  
jakarta-jetspeed-2/portals-bridges/frameworks/src/java/org/apache/portals/bridges/frameworks/Lookup.java
  
  Index: Lookup.java
  ===
  RCS file: 
/home/cvs/jakarta-jetspeed-2/portals-bridges/frameworks/src/java/org/apache/portals/bridges/frameworks/Lookup.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- Lookup.java   12 Nov 2004 06:28:56 -  1.1
  +++ Lookup.java   12 Nov 2004 19:35:46 -  1.2
  @@ -25,4 +25,5 @@
   public interface Lookup
   {
   boolean lookup(String key);
  +boolean isNew();
   }
  
  
  
  1.8   +33 -20
jakarta-jetspeed-2/portals-bridges/frameworks/src/java/org/apache/portals/bridges/frameworks/GenericFrameworkPortlet.java
  
  Index: GenericFrameworkPortlet.java
  ===
  RCS file: 
/home/cvs/jakarta-jetspeed-2/portals-bridges/frameworks/src/java/org/apache/portals/bridges/frameworks/GenericFrameworkPortlet.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -r1.7 -r1.8
  --- GenericFrameworkPortlet.java  12 Nov 2004 06:28:56 -  1.7
  +++ GenericFrameworkPortlet.java  12 Nov 2004 19:35:46 -  1.8
  @@ -56,6 +56,7 @@
*/
   public class GenericFrameworkPortlet extends GenericVelocityPortlet
   {
  +
   /**
* Init Parameter: default spring configuration property
*/
  @@ -69,11 +70,12 @@
   private static final String PREFS_SUFFIX = ".prefs";
   
   private static final String SESSION_ERROR_MESSAGES = 
"portals.bridges.framework.errors";
  +
   /**
* Action signature for calling velocity portlet actions
*/
   private static final Class[] VELOCITY_PORTLET_ACTION_SIGNATURE =
  -{ ActionRequest.class, ActionResponse.class};
  +{ ActionRequest.class, ActionResponse.class, Object.class};
   
   private static PortletApplicationModel model = null;
   
  @@ -87,7 +89,7 @@
   {
   model.setExternalSupport(map);
   }
  -
  +
   public void init(PortletConfig config) throws PortletException
   {
   super.init(config);
  @@ -141,7 +143,7 @@
   if (errors.isEmpty())
   {
   
request.getPortletSession().removeAttribute(SESSION_ERROR_MESSAGES, 
PortletSession.PORTLET_SCOPE);
  -
  +
   // (4) execute the velocity action
   String action = 
request.getParameter(FrameworkConstants.BRIDGES_ACTION);
   if (null == action)
  @@ -157,7 +159,7 @@
   else
   {
   // call the specified action in the post params
  -String actionForward = invokeVelocityPortletAction(action, 
request, response);
  +String actionForward = invokeVelocityPortletAction(action, 
request, response, bean);
   forward = model.getForward(actionForward);
   }
   }
  @@ -327,7 +329,8 @@
   if (view == null || view.equals(this.getDefaultViewPage()))
   {
   // clear it
  -
response.setRenderParameter(FrameworkConstants.VIEW_VIEW_MODE, view); 
//(String) null);
  +
response.setRenderParameter(FrameworkConstants.VIEW_VIEW_MODE, view); //(String)
  + 
 // null);
   }
   else
   {
  @@ -339,7 +342,8 @@
   if (view == null || view.equals(this.getDefaultEditPage()))
   {
   // clear it
  -
response.setRenderParameter(FrameworkConstants.VIEW_EDIT_MODE, view); 
//(String) null);
  +
response.setRenderParameter(FrameworkConstants.VIEW_EDIT_MODE, view); //(String)
  + 
 // null);
   }
   else
   {
  @@ -350,7 +354,8 @@
   {
   if (view == null || view.equals(this.getDefaultHelpPage()))
   {
  -
response.setRenderParameter(FrameworkConstants.VIEW_HELP_MODE, view); 
//(String) null);
  +
response.setRenderParameter(FrameworkConstants.VIEW_HELP_MODE, view); //(String)
  + 
 // null);
   }
   else
   {
  @@ -365,7 +370,8 @@
   if (vi

cvs commit: jakarta-jetspeed-2/components/web-content/src/java/org/apache/jetspeed/portlet IFrameGenericPortlet.java

[taylor]

taylor  2004/11/12 11:34:52

  Modified:components/web-content/src/java/org/apache/jetspeed/portlet
IFrameGenericPortlet.java
  Log:
  remove debug
  
  Revision  ChangesPath
  1.2   +1 -2  
jakarta-jetspeed-2/components/web-content/src/java/org/apache/jetspeed/portlet/IFrameGenericPortlet.java
  
  Index: IFrameGenericPortlet.java
  ===
  RCS file: 
/home/cvs/jakarta-jetspeed-2/components/web-content/src/java/org/apache/jetspeed/portlet/IFrameGenericPortlet.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- IFrameGenericPortlet.java 12 Nov 2004 06:30:40 -  1.1
  +++ IFrameGenericPortlet.java 12 Nov 2004 19:34:51 -  1.2
  @@ -95,7 +95,6 @@
   else
   value = getAttributePreference(prefs, attribute);
   
  -System.out.println("ATT = " + attribute + "VALUE = " + value);
   if (value == null || value == "") { return; }
   content.append(" 
").append(attribute).append("=\"").append(value).append("\"");
   }
  
  
  

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

cvs commit: jakarta-jetspeed-2/applications/pam/src/webapp/WEB-INF/view/site folder-details.vm

[taylor]

taylor  2004/11/12 11:34:32

  Modified:applications/pam/src/webapp/WEB-INF/view/site
folder-details.vm
  Log:
  *** keyword substitution change ***
  
  Revision  ChangesPath
  1.2   +21 -17
jakarta-jetspeed-2/applications/pam/src/webapp/WEB-INF/view/site/folder-details.vm
  
  Index: folder-details.vm
  ===
  RCS file: 
/home/cvs/jakarta-jetspeed-2/applications/pam/src/webapp/WEB-INF/view/site/folder-details.vm,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- folder-details.vm 12 Nov 2004 06:34:26 -  1.1
  +++ folder-details.vm 12 Nov 2004 19:34:32 -  1.2
  @@ -1,17 +1,21 @@
  -#set ($MESSAGES = $portletConfig.getResourceBundle($renderRequest.Locale))

  -

  -$MESSAGES.getString('folder.details')

  -

  -

  -

  -

  -

  -#formField('Title' "$!folderBean.Title" "40" 'title' $MESSAGES $ERRORS)

  -

  -

  -$MESSAGES.getString('add.page')

  -$MESSAGES.getString('add.subfolder')

  -$MESSAGES.getString('add.docset')

  -

  -

  -#ErrorMessages($ERRORS)

  +#set ($MESSAGES = $portletConfig.getResourceBundle($renderRequest.Locale))
  +
  +#if ($!folderBean.isNew())
  +
  +#else
  +$MESSAGES.getString('folder.details')
  +
  +
  +
  +
  +#formField('Title' "$!folderBean.Title" "40" 'title' $MESSAGES $ERRORS)
  +
  +
  +
  +$MESSAGES.getString('add.page')
  +$MESSAGES.getString('add.subfolder')
  +$MESSAGES.getString('add.docset')
  +
  +
  +#ErrorMessages($ERRORS)
  +#end
  \ No newline at end of file
  
  
  

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

cvs commit: jakarta-jetspeed-2/applications/pam/src/java/org/apache/jetspeed/portlets/site SiteBrowserPortlet.java FolderProxyBean.java

[taylor]

taylor  2004/11/12 11:34:07

  Modified:applications/pam/src/java/org/apache/jetspeed/portlets/site
SiteBrowserPortlet.java FolderProxyBean.java
  Log:
  - use the path as the unique identifier in the Site treeview
  - add support for isNew in the folder bean
  
  Revision  ChangesPath
  1.6   +3 -3  
jakarta-jetspeed-2/applications/pam/src/java/org/apache/jetspeed/portlets/site/SiteBrowserPortlet.java
  
  Index: SiteBrowserPortlet.java
  ===
  RCS file: 
/home/cvs/jakarta-jetspeed-2/applications/pam/src/java/org/apache/jetspeed/portlets/site/SiteBrowserPortlet.java,v
  retrieving revision 1.5
  retrieving revision 1.6
  diff -u -r1.5 -r1.6
  --- SiteBrowserPortlet.java   12 Nov 2004 06:34:25 -  1.5
  +++ SiteBrowserPortlet.java   12 Nov 2004 19:34:07 -  1.6
  @@ -204,8 +204,8 @@
   Iterator folderIter = childFolders.iterator();
   while (folderIter.hasNext())
   {
  -Folder childFolder = (Folder) folderIter.next();
  -TreeControlNode childNode = new 
TreeControlNode(childFolder.getName(), null, childFolder
  +Folder childFolder = (Folder) folderIter.next(); 
   
  +TreeControlNode childNode = new 
TreeControlNode(childFolder.getPath(), null, childFolder
   .getTitle(locale), 
PortletApplicationResources.PORTLET_URL, null, false, "FOLDER_DOMAIN");
   parent.addChild(childNode);
   buildFolderNodes(childFolder, childNode, locale);
  
  
  
  1.2   +17 -1 
jakarta-jetspeed-2/applications/pam/src/java/org/apache/jetspeed/portlets/site/FolderProxyBean.java
  
  Index: FolderProxyBean.java
  ===
  RCS file: 
/home/cvs/jakarta-jetspeed-2/applications/pam/src/java/org/apache/jetspeed/portlets/site/FolderProxyBean.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- FolderProxyBean.java  12 Nov 2004 06:34:25 -  1.1
  +++ FolderProxyBean.java  12 Nov 2004 19:34:07 -  1.2
  @@ -39,6 +39,7 @@
   private transient Folder folder;
   private String title;
   private String lookupKey;
  +private boolean isNew = true;
   
   public FolderProxyBean()
   {
  @@ -61,6 +62,7 @@
   {
   this.folder = pageManager.getFolder(key);
   setTitle(folder.getTitle());
  +isNew = false;
   }
   }
   catch (Exception ne)
  @@ -98,5 +100,19 @@
   public void setLookupKey(String lookupKey)
   {
   this.lookupKey = lookupKey;
  +}
  +/**
  + * @return Returns the isNew.
  + */
  +public boolean isNew()
  +{
  +return isNew;
  +}
  +/**
  + * @param isNew The isNew to set.
  + */
  +public void setNew(boolean isNew)
  +{
  +this.isNew = isNew;
   }
   }
  
  
  

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: portal project deployment in WEB-INF/classes vs. WEB-INF/lib

[David Sean Taylor]

Randy Watler wrote:
Gang,
The deployment of individual class files from the portal project in
WEB-INF/classes instead of packaging as a jar and deploying in WEB-INF/lib
is causing java Permission checks to be significantly slower than they could
be.
The reason for this is that each individual *.class file is being treated as
a separate CodeSource/PermissionDomain and must be tested individually when
AccessController.checkPermission() is invoked. All classes deployed in a jar
file reuse a single CodeSource/PermissionDomain. Minimizing the number of
CodeSources/PermissionDomains will limit the number of access to
RdbmsPolicy.getPolicies(), because each unique CodeSource on the call stack
between the SecurityValveImpl.invoke() and
AccessController.checkPermission() invocations will result in an additional
RdbmsPolicy.getPolicies() hit, (per each user/unique call stack).

Do any of you have a problem with me modifying the maven build to package
the class files into a single jar? Was there a reason to deploy in
WEB-INF/classes instead of WEB-INF/lib in the first place?
Any input or comments are appreciated. Thanks,
Randy
I think this is (or was) Maven's standard way of packaging a war file.
Im not opposed to changing it, but if you do, please make sure nothing 
else breaks (such as the hotdeploy goal)

Looking at RdbmsPolicy.getDefaultPolicy():
I
private static String defaultPolicy = "sun.security.provider.PolicyFile";
then
  Class policyClass = Class.forName(RdbmsPolicy.defaultPolicy);
return (Policy) policyClass.newInstance();
Is that where the performance hit is?
Or is it in getPermissons(). This code also looks suspect to me, 
especially the policy.refresh():

// TODO Is there a better way to do this?
// If the permission is not found here then delegate it
// to the standard java Policy class instance.
Policy.setPolicy(RdbmsPolicy.getDefaultPolicy());
Policy policy = Policy.getPolicy();
policy.refresh();
PermissionCollection defaultPerms = 
policy.getPermissions(codeSource);
// Revert back to the current policy.
Policy.setPolicy(this);
// Return the default permission collection.
return defaultPerms;

--
David Sean Taylor
Bluesunrise Software
[EMAIL PROTECTED]
[office] +01 707 773 4646
[mobile] +01 707 529 9194
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: applications re-org

[David Le Strat]

+1 
--- "Scott T. Weaver"
<[EMAIL PROTECTED]> wrote:

> +1
> David Sean Taylor wrote:
> 
> > in preparation for m1, propose to reorg the
> applications folder.
> > currently we have:
> >
> > + demo
> > + jsf-demo
> > + localeselector
> > + pam
> > + perl
> > + php
> > + security
> > + struts-demo
> >
> > into
> >
> > + demo
> > + pam
> > + security
> > + general (new)
> > + rss (xml, syndication) (new)
> >
> > where:
> >
> > php, perl, jsf-demo, struts-demo move to -> demo
> > localeselector, (and iframeportlet) move to ->
> general
> >
> > not sure if 'general' if the best name
> > im thinking of putting general (theres that word
> again) useful nondemo 
> > portlets such as localeselector, iframeportlet,
> calendar (coming 
> > soon), bookmark, weather, stockquote etc since
> they are not really 
> > demo so much as real useful portlets.
> >
> > for the rss pa, i plan to port the J1 rss, xslt
> and bbc portlets
> > other names for this pa could be xml or
> syndication
> >
> > suggestions objections welcome
> >
> 
> 
> -- 
> "Great minds discuss ideas. Average minds discuss
> events. Small minds discuss people."  - Admiral
> Hyman Rickover
> 
> ***
> *   Scott T. Weaver   *
> * <[EMAIL PROTECTED]> *
> * *
> * --  *
> *   Apache Jetspeed Enterprise Portal *
> * Apache Pluto Portlet Container  *
> * *
> * OpenEdit, Website Content Management*
> *    *
> ***
> 
> 
>
-
> To unsubscribe, e-mail:
> [EMAIL PROTECTED]
> For additional commands, e-mail:
> [EMAIL PROTECTED]
> 
> 




__ 
Do you Yahoo!? 
Check out the new Yahoo! Front Page. 
www.yahoo.com 
 


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

portal project deployment in WEB-INF/classes vs. WEB-INF/lib

[Randy Watler]

Gang,

The deployment of individual class files from the portal project in
WEB-INF/classes instead of packaging as a jar and deploying in WEB-INF/lib
is causing java Permission checks to be significantly slower than they could
be.

The reason for this is that each individual *.class file is being treated as
a separate CodeSource/PermissionDomain and must be tested individually when
AccessController.checkPermission() is invoked. All classes deployed in a jar
file reuse a single CodeSource/PermissionDomain. Minimizing the number of
CodeSources/PermissionDomains will limit the number of access to
RdbmsPolicy.getPolicies(), because each unique CodeSource on the call stack
between the SecurityValveImpl.invoke() and
AccessController.checkPermission() invocations will result in an additional
RdbmsPolicy.getPolicies() hit, (per each user/unique call stack).

Do any of you have a problem with me modifying the maven build to package
the class files into a single jar? Was there a reason to deploy in
WEB-INF/classes instead of WEB-INF/lib in the first place?

Any input or comments are appreciated. Thanks,

Randy

Re: applications re-org

[Roger Ruttimann]

+1 for the reorg of applications
Why not just using portlets instead of general. In a subfolder it could 
be grouped by functionality such as webcontent (iframe,..), utilities 
(localselector, calendar,..).

David Sean Taylor wrote:
in preparation for m1, propose to reorg the applications folder.
currently we have:
+ demo
+ jsf-demo
+ localeselector
+ pam
+ perl
+ php
+ security
+ struts-demo
into
+ demo
+ pam
+ security
+ general (new)
+ rss (xml, syndication) (new)
where:
php, perl, jsf-demo, struts-demo move to -> demo
localeselector, (and iframeportlet) move to -> general
not sure if 'general' if the best name
im thinking of putting general (theres that word again) useful nondemo 
portlets such as localeselector, iframeportlet, calendar (coming 
soon), bookmark, weather, stockquote etc since they are not really 
demo so much as real useful portlets.

for the rss pa, i plan to port the J1 rss, xslt and bbc portlets
other names for this pa could be xml or syndication
suggestions objections welcome

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: applications re-org

[Scott T. Weaver]

+1
David Sean Taylor wrote:
in preparation for m1, propose to reorg the applications folder.
currently we have:
+ demo
+ jsf-demo
+ localeselector
+ pam
+ perl
+ php
+ security
+ struts-demo
into
+ demo
+ pam
+ security
+ general (new)
+ rss (xml, syndication) (new)
where:
php, perl, jsf-demo, struts-demo move to -> demo
localeselector, (and iframeportlet) move to -> general
not sure if 'general' if the best name
im thinking of putting general (theres that word again) useful nondemo 
portlets such as localeselector, iframeportlet, calendar (coming 
soon), bookmark, weather, stockquote etc since they are not really 
demo so much as real useful portlets.

for the rss pa, i plan to port the J1 rss, xslt and bbc portlets
other names for this pa could be xml or syndication
suggestions objections welcome

--
"Great minds discuss ideas. Average minds discuss events. Small minds discuss 
people."  - Admiral Hyman Rickover
***
*   Scott T. Weaver   *
* <[EMAIL PROTECTED]> *
* *
* --  *
*   Apache Jetspeed Enterprise Portal *
* Apache Pluto Portlet Container  *
* *
* OpenEdit, Website Content Management*
*    *
***
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]