You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jd...@apache.org on 2008/09/28 10:42:19 UTC
svn commit: r699785 - in /geronimo/gshell/trunk:
gshell-remote/gshell-remote-common/src/main/java/org/apache/geronimo/gshell/remote/jaas/
gshell-support/gshell-artifact/src/main/java/org/apache/geronimo/gshell/artifact/
Author: jdillon
Date: Sun Sep 28 01:42:18 2008
New Revision: 699785
URL: http://svn.apache.org/viewvc?rev=699785&view=rev
Log:
Add some todo's least I forget about them
Modified:
geronimo/gshell/trunk/gshell-remote/gshell-remote-common/src/main/java/org/apache/geronimo/gshell/remote/jaas/JaasConfigurationLoader.java
geronimo/gshell/trunk/gshell-support/gshell-artifact/src/main/java/org/apache/geronimo/gshell/artifact/DefaultArtifactManager.java
geronimo/gshell/trunk/gshell-support/gshell-artifact/src/main/java/org/apache/geronimo/gshell/artifact/GShellArtifactMetadataSource.java
Modified: geronimo/gshell/trunk/gshell-remote/gshell-remote-common/src/main/java/org/apache/geronimo/gshell/remote/jaas/JaasConfigurationLoader.java
URL: http://svn.apache.org/viewvc/geronimo/gshell/trunk/gshell-remote/gshell-remote-common/src/main/java/org/apache/geronimo/gshell/remote/jaas/JaasConfigurationLoader.java?rev=699785&r1=699784&r2=699785&view=diff
==============================================================================
--- geronimo/gshell/trunk/gshell-remote/gshell-remote-common/src/main/java/org/apache/geronimo/gshell/remote/jaas/JaasConfigurationLoader.java (original)
+++ geronimo/gshell/trunk/gshell-remote/gshell-remote-common/src/main/java/org/apache/geronimo/gshell/remote/jaas/JaasConfigurationLoader.java Sun Sep 28 01:42:18 2008
@@ -35,6 +35,10 @@
public class JaasConfigurationLoader
implements BeanClassLoaderAware
{
+ //
+ // TODO: Look at using JSecurity or spring-security to handle auth and such?
+ //
+
private static final String KEY = "java.security.auth.login.config";
private final Logger log = LoggerFactory.getLogger(getClass());
Modified: geronimo/gshell/trunk/gshell-support/gshell-artifact/src/main/java/org/apache/geronimo/gshell/artifact/DefaultArtifactManager.java
URL: http://svn.apache.org/viewvc/geronimo/gshell/trunk/gshell-support/gshell-artifact/src/main/java/org/apache/geronimo/gshell/artifact/DefaultArtifactManager.java?rev=699785&r1=699784&r2=699785&view=diff
==============================================================================
--- geronimo/gshell/trunk/gshell-support/gshell-artifact/src/main/java/org/apache/geronimo/gshell/artifact/DefaultArtifactManager.java (original)
+++ geronimo/gshell/trunk/gshell-support/gshell-artifact/src/main/java/org/apache/geronimo/gshell/artifact/DefaultArtifactManager.java Sun Sep 28 01:42:18 2008
@@ -43,6 +43,10 @@
{
private final Logger log = LoggerFactory.getLogger(getClass());
+ //
+ // TODO: Look at implementing this with mercury
+ //
+
@Requirement
private ArtifactFactory artifactFactory;
Modified: geronimo/gshell/trunk/gshell-support/gshell-artifact/src/main/java/org/apache/geronimo/gshell/artifact/GShellArtifactMetadataSource.java
URL: http://svn.apache.org/viewvc/geronimo/gshell/trunk/gshell-support/gshell-artifact/src/main/java/org/apache/geronimo/gshell/artifact/GShellArtifactMetadataSource.java?rev=699785&r1=699784&r2=699785&view=diff
==============================================================================
--- geronimo/gshell/trunk/gshell-support/gshell-artifact/src/main/java/org/apache/geronimo/gshell/artifact/GShellArtifactMetadataSource.java (original)
+++ geronimo/gshell/trunk/gshell-support/gshell-artifact/src/main/java/org/apache/geronimo/gshell/artifact/GShellArtifactMetadataSource.java Sun Sep 28 01:42:18 2008
@@ -43,6 +43,10 @@
{
private final Logger log = LoggerFactory.getLogger(getClass());
+ //
+ // TODO: Look into creating a simple model that only deals with dependency bits and perform basic processing, avoiding the need for all this Maven stuff
+ //
+
@Requirement(hint="maven")
private ArtifactMetadataSource delegate;