lita-activedirectory 0.2.3 → 0.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8de315c678f7820e8cb6191fa6879d8039d6b98f
-  data.tar.gz: 217e82c55ddb9258aefaa348bf25995bfd568b5c
+  metadata.gz: 08485dbe161fadc4b33a407554004bef9abdaaa6
+  data.tar.gz: eef09887231db2d23732984911374d6045efc453
 SHA512:
-  metadata.gz: 758c3742af13a513eb65d6e0a8f1f7f0e90c2bd187ad44e167a2968567b096131768bf43bd70be06a82b45adcb57b56ca4ff5044a1539d7e1c00ae521d13eee7
-  data.tar.gz: 53a6ad9ee77b474447769f2e163134e3c616bd721f1ff5eb354f70e95206f49a05a329d4783f2178f81cecd9841e4da2211096d906f1b355c19a64edf440668c
+  metadata.gz: f4f758acc794da870b4a9a6038ea826c8e254fca42e1e32fda16b0b98d1b5747d0cc227b0a4bed609b2c6480a9780a267dc6b470c90807ac2b35643bc20ed94d
+  data.tar.gz: eebbe685f1433fdd77886369fac3c02f7ec64c3265ab27abd1ceaa6d89646e2d77a227729c34a40fe1c307c3c337d9e762426a44454e874c6883687610ed8988

data/lib/lita/handlers/activedirectory.rb

@@ -6,6 +6,7 @@ module Lita
       config :port, required: true, type: Integer, default: 389
       config :basedn, required: true, type: String
       config :user_basedn, required: true, type: String
+      config :group_basedn, required: true, type: String
       config :username, required: true, type: String
       config :password, required: true, type: String
 
@@ -53,7 +54,13 @@ module Lita
         user = response.matches[0][0]
         response.reply_with_mention(t('replies.user_groups.working'))
         group_results = user_groups_query(user)
-        handle_user_group_query(response, user, group_results)
+        if group_results.nil?
+          response.reply_with_mention(
+            t('replies.user_groups.error', user: user)
+          )
+        else
+          response.reply group_results
+        end
       end
 
       private
@@ -75,23 +82,6 @@ module Lita
         end
       end
 
-      def handle_user_query(response, user, result)
-        case result
-        when true
-          response.reply_with_mention(
-            t('replies.user_locked?.locked', user: user)
-          )
-        when false
-          response.reply_with_mention(
-            t('replies.user_locked?.notlocked', user: user)
-          )
-        when nil
-          response.reply_with_mention(
-            t('replies.user_locked?.error', user: user)
-          )
-        end
-      end
-
       def handle_unlock_query(response, user, result)
         case result
         when true

data/lib/utils/cratususer.rb

@@ -32,7 +32,7 @@ module Utils
         nil
       end
       groups = user.member_of
-      groups.each(&:name)
+      groups.map(&:name).join("\n")
     end
 
     def unlock_user(username)

data/lita-activedirectory.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = 'lita-activedirectory'
-  spec.version       = '0.2.3'
+  spec.version       = '0.2.4'
   spec.authors       = ['Daniel Schaaff']
   spec.email         = ['dschaaff@knuedge.com']
   spec.description   = 'ldap/active directory instructions for Lita'

data/locales/en.yml

@@ -10,7 +10,7 @@ en:
             syntax: unlock <username>
             desc: unlocks an active directory user account
           user_groups:
-            syntax: <username> user_groups
+            syntax: <username> groups
             desc: lists all group memberships for the given username
         replies:
           user_locked?:

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lita-activedirectory
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.2.4
 platform: ruby
 authors:
 - Daniel Schaaff