You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@esme.apache.org by le...@apache.org on 2011/05/16 00:58:56 UTC

svn commit: r1103572 - in /esme/trunk/server/src/main: resources/props/default.props scala/org/apache/esme/model/UserAuth.scala webapp/scripts/display_single_message.js webapp/templates-hidden/default.html webapp/templates-hidden/upw_login_form.html

Author: lester
Date: Sun May 15 22:58:56 2011
New Revision: 1103572

URL: http://svn.apache.org/viewvc?rev=1103572&view=rev
Log:
Made other types of auth inactive on login screen when LDAP is enabled. Externalized allowed role list into property file.

Modified:
    esme/trunk/server/src/main/resources/props/default.props
    esme/trunk/server/src/main/scala/org/apache/esme/model/UserAuth.scala
    esme/trunk/server/src/main/webapp/scripts/display_single_message.js
    esme/trunk/server/src/main/webapp/templates-hidden/default.html
    esme/trunk/server/src/main/webapp/templates-hidden/upw_login_form.html

Modified: esme/trunk/server/src/main/resources/props/default.props
URL: http://svn.apache.org/viewvc/esme/trunk/server/src/main/resources/props/default.props?rev=1103572&r1=1103571&r2=1103572&view=diff
==============================================================================
--- esme/trunk/server/src/main/resources/props/default.props (original)
+++ esme/trunk/server/src/main/resources/props/default.props Sun May 15 22:58:56 2011
@@ -54,3 +54,7 @@ ldap.uidPrefix=uid
 ldap.userBase=ou=Users,ou=esme,dc=lester,dc=org
 ;Group base DN to check whether user has specific role
 ldap.groupBase=ou=Groups,ou=esme,dc=lester,dc=org
+
+
+;Allow access to application for following roles
+role_list=esme-users,monitoring-admin
\ No newline at end of file

Modified: esme/trunk/server/src/main/scala/org/apache/esme/model/UserAuth.scala
URL: http://svn.apache.org/viewvc/esme/trunk/server/src/main/scala/org/apache/esme/model/UserAuth.scala?rev=1103572&r1=1103571&r2=1103572&view=diff
==============================================================================
--- esme/trunk/server/src/main/scala/org/apache/esme/model/UserAuth.scala (original)
+++ esme/trunk/server/src/main/scala/org/apache/esme/model/UserAuth.scala Sun May 15 22:58:56 2011
@@ -20,6 +20,7 @@
 package org.apache.esme.model
 
 import net.liftweb._
+import common.Box._
 import common.Logger._
 import mapper._
 import openid._
@@ -120,8 +121,13 @@ trait FieldSet {
 }
 
 object UserPwdAuthModule extends AuthModule {
-  def loginPresentation: Box[NodeSeq] =
-  TemplateFinder.findAnyTemplate("templates-hidden" :: "upw_login_form" :: Nil)
+  def loginPresentation: Box[NodeSeq] = {
+    val ldapBind : CssBindFunc = "#ldapEnabled [value]" #> (Props.getBool("ldap.enabled") openOr false)
+    TemplateFinder.findAnyTemplate("templates-hidden" :: "upw_login_form" :: Nil) match {
+      case Full(tpl) => Full(ldapBind(tpl))
+      case _ => Empty
+    }
+  }
 
   def moduleName: String = "upw"
 
@@ -281,6 +287,11 @@ trait LDAPBase {
 
   object myLdapVendor extends LDAPVendor
 
+  val rolesToCheck = Props.get("role_list") match {
+    case Full(s) => s.split(',').toList
+    case _ => Nil
+  }
+
   var currentRole : String = _
 
   def myLdap : LDAPVendor = {
@@ -358,12 +369,6 @@ trait LDAPBase {
 
 object ContainerManagedAuthModule extends AuthModule with LDAPBase {
 
-  // It's possible to get roles list from some external source
-  // for example from LDAP via Lift API
-  val rolesToCheck = List(
-    "esme-users"
-  )
-
   override def isDefault = false
 
   def loginPresentation: Box[NodeSeq] = Empty
@@ -443,12 +448,6 @@ object ContainerManagedAuthModule extend
 
 object LDAPAuthModule extends AuthModule with LDAPBase {
 
-  // It's possible to get roles list from some external source
-  // for example from LDAP via Lift API
-  val rolesToCheck = List(
-    "esme-users", "monitoring-admin"
-  )
-
   override def isDefault = false
 
   def loginPresentation: Box[NodeSeq] = TemplateFinder.findAnyTemplate("templates-hidden" :: "ldap_login_form" :: Nil)

Modified: esme/trunk/server/src/main/webapp/scripts/display_single_message.js
URL: http://svn.apache.org/viewvc/esme/trunk/server/src/main/webapp/scripts/display_single_message.js?rev=1103572&r1=1103571&r2=1103572&view=diff
==============================================================================
--- esme/trunk/server/src/main/webapp/scripts/display_single_message.js (original)
+++ esme/trunk/server/src/main/webapp/scripts/display_single_message.js Sun May 15 22:58:56 2011
@@ -24,10 +24,12 @@ function login()
   	
   if (document.forms[0].open_id.value != "")
      document.forms[0].action = 'open_id/login';
- else
+  else if(jQuery('input#ldapEnabled').val() == 'true')
+     document.forms[0].action = 'ldap/login';
+  else
      document.forms[0].action = 'authentication/login';
-    
-     
+
+
   document.forms[0].submit();	
 }                      
 // ]]>
\ No newline at end of file

Modified: esme/trunk/server/src/main/webapp/templates-hidden/default.html
URL: http://svn.apache.org/viewvc/esme/trunk/server/src/main/webapp/templates-hidden/default.html?rev=1103572&r1=1103571&r2=1103572&view=diff
==============================================================================
--- esme/trunk/server/src/main/webapp/templates-hidden/default.html (original)
+++ esme/trunk/server/src/main/webapp/templates-hidden/default.html Sun May 15 22:58:56 2011
@@ -37,7 +37,7 @@
                     </div>
                     <div id="top-menu">
                         <lift:Menu.item name="sign_up">
-                       <img src="images/btn-signup.gif" alt="" />
+                       <img id="btn_signup" src="images/btn-signup.gif" alt="" />
                     </lift:Menu.item>
                     </div>
                 </div>

Modified: esme/trunk/server/src/main/webapp/templates-hidden/upw_login_form.html
URL: http://svn.apache.org/viewvc/esme/trunk/server/src/main/webapp/templates-hidden/upw_login_form.html?rev=1103572&r1=1103571&r2=1103572&view=diff
==============================================================================
--- esme/trunk/server/src/main/webapp/templates-hidden/upw_login_form.html (original)
+++ esme/trunk/server/src/main/webapp/templates-hidden/upw_login_form.html Sun May 15 22:58:56 2011
@@ -25,6 +25,10 @@
             jQuery('input[type=\'image\']').bind('click', function() {
                 jQuery('form[name=\'loginFrm\']').attr('action', 'javascript:login()').submit();
             });
+            if(jQuery('input#ldapEnabled').val() == 'true') {
+                jQuery('div#openIdSection').hide();
+                jQuery('img#btn_signup').hide();
+            }
         });
         </script>
  
@@ -49,9 +53,11 @@
 							<input type="password" name="password" value="" class="inputBox2"/><!--narrow input field-->
 						</div>	
 						
+                        <input id="ldapEnabled" type="hidden" value=""/>
+
 						<div class="line"></div>
 						
-						<div class="post-form-row"> 
+						<div id="openIdSection" class="post-form-row">
 							<label><lift:loc>ui_login_openid</lift:loc></label>
 							<input type="text" name="open_id" value="" class="inputBox2"/><!--narrow input field-->
 						</div>