secure 0.1.0 → 0.1.1

data/lib/secure/child_process.rb

@@ -3,7 +3,7 @@ module Secure
     def initialize(opts, read_file, write_file)
       read_file.close
       @pipe = write_file
-      @timeout = opts[:timeout] || 1
+      @timeout = opts[:timeout]
     end
 
     def guard_threads
@@ -15,14 +15,16 @@ module Secure
         $SAFE=3
         yield
       end
-
-      guard_threads << GuardThread.kill_thread_on_timeout(@timeout, thread)
-
+      decorate_with_guard_threads(thread)
       Response.success(thread.value)
     rescue Exception => e
       Response.error(e)
     end
 
+    def decorate_with_guard_threads(thread)
+      GuardThread.kill_thread_on_timeout(@timeout, thread) if @timeout
+    end
+
     def execute
       ret = safely_run_block { yield }
       @pipe.write(Marshal.dump(ret))

data/lib/secure/runner.rb

@@ -11,7 +11,7 @@ module Secure
         begin
           ChildProcess.new(@opts, read_file, write_file).execute { yield }
         ensure
-          exit!
+          exit
         end
       end
 

data/lib/secure/version.rb

@@ -1,3 +1,3 @@
 module Secure
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

data/spec/secure_spec.rb

@@ -91,4 +91,15 @@ describe Secure do
       response.should be_success
     end
   end
+
+  context "error information" do
+    it "should know where the syntax is invalid" do
+      string = "while true; end; end"
+      response = Secure.ly({}, string) do |string|
+        eval(string)
+      end
+      response.should_not be_success
+      response.error.should be_a(SyntaxError)
+    end
+  end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: secure
 version: !ruby/object:Gem::Version 
-  hash: 27
+  hash: 25
   prerelease: 
   segments: 
   - 0
   - 1
-  - 0
-  version: 0.1.0
+  - 1
+  version: 0.1.1
 platform: ruby
 authors: 
 - Tejas Dinkar