[Groonga-commit] droonga/fluent-plugin-droonga at 204175d [master] Reject subscription without route

Back to archive index

YUKI Hiroshi null+****@clear*****
Thu Oct 31 18:03:03 JST 2013


YUKI Hiroshi	2013-10-31 18:03:03 +0900 (Thu, 31 Oct 2013)

  New Revision: 204175d3290f57979d0768fbd83c4fd60ffb2ad4
  https://github.com/droonga/fluent-plugin-droonga/commit/204175d3290f57979d0768fbd83c4fd60ffb2ad4

  Message:
    Reject subscription without route

  Modified files:
    lib/droonga/watcher.rb

  Modified: lib/droonga/watcher.rb (+3 -1)
===================================================================
--- lib/droonga/watcher.rb    2013-10-31 17:51:06 +0900 (fc1c6b1)
+++ lib/droonga/watcher.rb    2013-10-31 18:03:03 +0900 (79a738d)
@@ -33,7 +33,9 @@ module Droonga
       query      = request[:query]
       route      = request[:route]
 
-      if subscriber.nil? || subscriber.empty? || condition.nil? || query.nil?
+      # XXX better validation and error class must be written!!
+      if subscriber.nil? || subscriber.empty? || condition.nil? ||
+         query.nil? || route.nil?
         raise "invalid request"
       end
       raise "too long query" if query.size > 4095
-------------- next part --------------
HTML����������������������������...
Download 



More information about the Groonga-commit mailing list
Back to archive index