build-buddy 1.4.0 → 1.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 329f4a21f34dfda0663a97f4d74b691bba541491
-  data.tar.gz: 7f3da88951d7fd2a1a7bd982cc324eb550a82e0d
+  metadata.gz: 900f5709de49334bd3cb12ed7f221746734d80ab
+  data.tar.gz: f229e6ce3555005f977162275bfa627ed6f4ed5e
 SHA512:
-  metadata.gz: 0608f25926cd5646ea0665b86e53d1a5a1aa18079504a9d166dc99fb8ab85cd3c21e34f47566a8504bd6f8f598ef5569e67bea6112e901124ad6249735f20665
-  data.tar.gz: 5d5e776f0f9115d2636f3f8095132d8da9316e4d33acb39b208a2fc6ebca5f8016d926fead214a012670e784a438b0dbb8652829d2aaeeb8d746613d749ab37f
+  metadata.gz: fa2d4cc73ebed8ba46a5162dd8f5587bc08c7959e43fdf064a5de3669fcd8385ceefce1f7dcb6749bf917ad67398d44d30677c26db8d2dc7202612ccb3ed08ae
+  data.tar.gz: 74603720ddb9efe8bd522282417db16e1a26fcb31f1ce05111fc0689e7472cca635836b24cff37ecb4d16fa6b0a89ccab4fe77a31acc96da42f79b545292dd2f

data/lib/build_buddy.rb

@@ -8,5 +8,5 @@ require 'build_buddy/server'
 require 'build_buddy/build_data'
 
 module BuildBuddy
-  VERSION = "1.4.0"
+  VERSION = "1.4.1"
 end

data/lib/build_buddy/slacker.rb

@@ -22,7 +22,6 @@ module BuildBuddy
       end
       @rt_client.start_async
       @notify_slack_channel = nil
-      @reverse_user_map = nil
     end
 
     def on_slack_error(error)
@@ -32,9 +31,8 @@ module BuildBuddy
 
     def on_slack_hello
       user_id = @rt_client.self['id']
-      user_map = @rt_client.users.map {|user| [user['name'], user['id']]}.to_h
-      @reverse_user_map = user_map.invert
-      info "Connected to Slack as user id #{user_id} (@#{@reverse_user_map[user_id]})"
+      map_user_id_to_name = @rt_client.users.map {|user| [user['id'], user['name']]}.to_h
+      info "Connected to Slack as user id #{user_id} (@#{map_user_id_to_name[user_id]})"
 
       channel_map = @rt_client.channels.map {|channel| [channel['name'], channel['id']]}.to_h
       group_map = @rt_client.groups.map {|group| [group['name'], group['id']]}.to_h
@@ -66,7 +64,13 @@ module BuildBuddy
         end
         sending_user_name = data['username']
       else
-        sending_user_name = @reverse_user_map[sending_user_id]
+        map_user_id_to_name = @rt_client.users.map {|user| [user['id'], user['name']]}.to_h
+        sending_user_name = map_user_id_to_name[sending_user_id]
+
+        if sending_user_name.nil?
+          error "User #{sending_user_id} is not known"
+          return
+        end
       end
 
       # Don't respond if _we_ sent the message!

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: build-buddy
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 1.4.1
 platform: ruby
 authors:
 - John Lyon-smith
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-06 00:00:00.000000000 Z
+date: 2016-02-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: timers