- raise an exception if you try to set @timestamp to an invalid value

type. OH STATIC TYPING SOMETIMES I MISS YOU.

lib/logstash/event.rb

@@ -119,6 +119,10 @@ class LogStash::Event
   
   public
   def []=(str, value)
+    if str == "@timestamp" && !value.is_a?(Time)
+      raise TypeError, "The field '@timestamp' must be a Time, not a #{value.class} (#{value})"
+    end
+
     r = LogStash::Util::FieldReference.exec(str, @data) do |obj, key|
       obj[key] = value
     end

spec/event.rb

@@ -25,6 +25,12 @@ describe LogStash::Event do
     )
   end
 
+  context "[]=" do
+    it "should raise an exception if you attempt to set @timestamp to a value type other than a Time object" do
+      insist { subject["@timestamp"] = "crash!" }.raises(TypeError)
+    end
+  end
+
   context "#sprintf" do
     it "should report a unix timestamp for %{+%s}" do
       insist { subject.sprintf("%{+%s}") } == "1356998400"