microsandbox-rb 0.5.9 → 0.5.10

data/lib/microsandbox/exec_handle.rb

@@ -22,25 +22,28 @@ module Microsandbox
 
     # @return [String, nil] {#data} decoded as UTF-8 (lenient)
     def text
-      @data && @data.dup.force_encoding(Encoding::UTF_8)
+      @data&.dup&.force_encoding(Encoding::UTF_8)
     end
 
-    def started?      = @type == :started
-    def stdout?       = @type == :stdout
-    def stderr?       = @type == :stderr
-    def exited?       = @type == :exited
-    def failed?       = @type == :failed
-    def stdin_error?  = @type == :stdin_error
+    def started? = @type == :started
+    def stdout? = @type == :stdout
+    def stderr? = @type == :stderr
+    def exited? = @type == :exited
+    def failed? = @type == :failed
+    def stdin_error? = @type == :stdin_error
 
     def inspect
-      "#<Microsandbox::ExecEvent type=#{@type}#{@pid ? " pid=#{@pid}" : ""}" \
-        "#{@code ? " code=#{@code}" : ""}#{@data ? " data=#{@data.bytesize}B" : ""}>"
+      "#<Microsandbox::ExecEvent type=#{@type}#{" pid=#{@pid}" if @pid}" \
+        "#{" code=#{@code}" if @code}#{" data=#{@data.bytesize}B" if @data}>"
     end
   end
 
-  # The terminal status of a streamed execution, from {ExecHandle#wait}.
+  # The terminal status of a streamed execution ({ExecHandle#wait}) or of a
+  # sandbox process ({Sandbox#wait}, {Sandbox#stop_and_wait}).
   class ExitStatus
-    # @return [Integer]
+    # @return [Integer, nil] the exit code, or nil when the process was
+    #   terminated by a signal and so carries no code (e.g. a SIGKILL'd sandbox
+    #   from {Sandbox#kill} then {Sandbox#wait}) — check {#success?} in that case.
     attr_reader :exit_code
 
     def initialize(data)

data/lib/microsandbox/fs.rb

@@ -30,9 +30,9 @@ module Microsandbox
       @modified_ms && Time.at(@modified_ms / 1000.0)
     end
 
-    def file?       = @type == :file
-    def directory?  = @type == :directory
-    def symlink?    = @type == :symlink
+    def file? = @type == :file
+    def directory? = @type == :directory
+    def symlink? = @type == :symlink
 
     def inspect
       "#<Microsandbox::FsEntry path=#{@path.inspect} type=#{@type} size=#{@size}>"
@@ -57,10 +57,10 @@ module Microsandbox
       @created_ms = data["created_ms"]
     end
 
-    def readonly?   = @readonly
-    def file?       = @type == :file
-    def directory?  = @type == :directory
-    def symlink?    = @type == :symlink
+    def readonly? = @readonly
+    def file? = @type == :file
+    def directory? = @type == :directory
+    def symlink? = @type == :symlink
 
     # @return [Time, nil] last-modified time, if known
     def modified

data/lib/microsandbox/image.rb

@@ -56,7 +56,7 @@ module Microsandbox
   # The result of {Image.prune}.
   class ImagePruneReport
     attr_reader :image_refs_removed, :manifests_removed, :layers_removed,
-                :fsmeta_removed, :vmdk_removed, :bytes_reclaimed
+      :fsmeta_removed, :vmdk_removed, :bytes_reclaimed
 
     def initialize(data)
       @image_refs_removed = data["image_refs_removed"]

data/lib/microsandbox/network.rb

@@ -16,23 +16,23 @@ module Microsandbox
     module_function
 
     # Match any destination.
-    def any = { "destination_kind" => "any" }
+    def any = {"destination_kind" => "any"}
 
     # A single IP address (stored as a /32 or /128).
-    def ip(value) = { "destination_kind" => "ip", "destination" => value.to_s }
+    def ip(value) = {"destination_kind" => "ip", "destination" => value.to_s}
 
     # An IP network in CIDR notation (e.g. "10.0.0.0/8").
-    def cidr(value) = { "destination_kind" => "cidr", "destination" => value.to_s }
+    def cidr(value) = {"destination_kind" => "cidr", "destination" => value.to_s}
 
     # An exact domain name (matched against the resolved-hostname cache / SNI).
-    def domain(value) = { "destination_kind" => "domain", "destination" => value.to_s }
+    def domain(value) = {"destination_kind" => "domain", "destination" => value.to_s}
 
     # A domain suffix — matches the apex and any subdomain (e.g. ".internal").
-    def domain_suffix(value) = { "destination_kind" => "domain_suffix", "destination" => value.to_s }
+    def domain_suffix(value) = {"destination_kind" => "domain_suffix", "destination" => value.to_s}
 
     # A predefined group: :public, :loopback, :private, :link_local, :metadata,
     # :multicast, or :host.
-    def group(value) = { "destination_kind" => "group", "destination" => value.to_s.tr("_", "-") }
+    def group(value) = {"destination_kind" => "group", "destination" => value.to_s.tr("_", "-")}
   end
 
   # Factory for a single network-policy **rule**. A rule pairs an action
@@ -64,7 +64,7 @@ module Microsandbox
 
     # @api private
     def build(action, destination, direction, protocol, protocols, port, ports)
-      rule = { "action" => action, "direction" => direction.to_s }
+      rule = {"action" => action, "direction" => direction.to_s}
       rule.merge!(normalize_destination(destination))
       protos = (Array(protocols) + Array(protocol)).compact.map(&:to_s)
       rule["protocols"] = protos unless protos.empty?
@@ -78,7 +78,7 @@ module Microsandbox
       case dest
       when nil then {}
       when Hash then dest.each_with_object({}) { |(k, v), a| a[k.to_s] = v }
-      when String, Symbol then { "destination" => dest.to_s }
+      when String, Symbol then {"destination" => dest.to_s}
       else raise ArgumentError, "invalid rule destination: #{dest.inspect}"
       end
     end
@@ -149,7 +149,7 @@ module Microsandbox
       # @param deny_domain_suffixes [Array<String>] domain suffixes to deny
       # @return [NetworkPolicy]
       def custom(default_egress: :deny, default_ingress: :allow, rules: [],
-                 deny_domains: [], deny_domain_suffixes: [])
+        deny_domains: [], deny_domain_suffixes: [])
         h = {}
         h["default_egress"] = action_str(default_egress) unless default_egress.nil?
         h["default_ingress"] = action_str(default_ingress) unless default_ingress.nil?
@@ -163,12 +163,12 @@ module Microsandbox
       def coerce(network)
         case network
         when NetworkPolicy then network.to_h
-        when String, Symbol then { "preset" => canonical_preset(network) }
+        when String, Symbol then {"preset" => canonical_preset(network)}
         when Hash then from_hash(network)
         else
           raise ArgumentError,
-                "network: expects a preset name, a Microsandbox::NetworkPolicy, or a Hash " \
-                "(got #{network.class})"
+            "network: expects a preset name, a Microsandbox::NetworkPolicy, or a Hash " \
+            "(got #{network.class})"
         end
       end
 
@@ -186,11 +186,11 @@ module Microsandbox
         if sym.key?(:preset)
           if sym.key?(:rules) || sym.key?(:default_egress) || sym.key?(:default_ingress)
             raise ArgumentError,
-                  "network preset: cannot be combined with rules:/default_egress:/" \
-                  "default_ingress: (the preset already defines its rules and defaults); " \
-                  "only deny_domains:/deny_domain_suffixes: may be layered on a preset"
+              "network preset: cannot be combined with rules:/default_egress:/" \
+              "default_ingress: (the preset already defines its rules and defaults); " \
+              "only deny_domains:/deny_domain_suffixes: may be layered on a preset"
           end
-          h = { "preset" => canonical_preset(sym[:preset]) }
+          h = {"preset" => canonical_preset(sym[:preset])}
           add_deny_lists(h, sym[:deny_domains], sym[:deny_domain_suffixes])
           h
         elsif sym.key?(:rules) || sym.key?(:default_egress) || sym.key?(:default_ingress)
@@ -213,7 +213,7 @@ module Microsandbox
           ds = Array(sym[:deny_domain_suffixes]).map(&:to_s)
           return {} if dd.empty? && ds.empty?
 
-          h = { "default_egress" => "allow", "default_ingress" => "allow" }
+          h = {"default_egress" => "allow", "default_ingress" => "allow"}
           add_deny_lists(h, dd, ds)
           h
         end
@@ -233,8 +233,8 @@ module Microsandbox
         key = name.to_s.downcase
         PRESET_ALIASES[key] ||
           raise(ArgumentError,
-                "unknown network preset #{name.inspect} " \
-                "(expected one of public_only/none/allow_all/non_local)")
+            "unknown network preset #{name.inspect} " \
+            "(expected one of public_only/none/allow_all/non_local)")
       end
 
       def action_str(action)

data/lib/microsandbox/patch.rb

@@ -31,7 +31,7 @@ module Microsandbox
     # @param replace [Boolean] allow shadowing a path already in the rootfs
     # @return [Hash]
     def text(path, content, mode: nil, replace: false)
-      h = { "kind" => "text", "path" => path.to_s, "content" => content.to_s, "replace" => replace ? true : false }
+      h = {"kind" => "text", "path" => path.to_s, "content" => content.to_s, "replace" => replace ? true : false}
       h["mode"] = Integer(mode) unless mode.nil?
       h
     end
@@ -43,7 +43,7 @@ module Microsandbox
     # @param replace [Boolean]
     # @return [Hash]
     def file(path, content, mode: nil, replace: false)
-      h = { "kind" => "file", "path" => path.to_s, "content" => content.to_s, "replace" => replace ? true : false }
+      h = {"kind" => "file", "path" => path.to_s, "content" => content.to_s, "replace" => replace ? true : false}
       h["mode"] = Integer(mode) unless mode.nil?
       h
     end
@@ -52,7 +52,7 @@ module Microsandbox
     # lower image layer is copied up first, then appended.
     # @return [Hash]
     def append(path, content)
-      { "kind" => "append", "path" => path.to_s, "content" => content.to_s }
+      {"kind" => "append", "path" => path.to_s, "content" => content.to_s}
     end
 
     # Copy a host file into the rootfs.
@@ -62,7 +62,7 @@ module Microsandbox
     # @param replace [Boolean]
     # @return [Hash]
     def copy_file(src, dst, mode: nil, replace: false)
-      h = { "kind" => "copy_file", "src" => src.to_s, "dst" => dst.to_s, "replace" => replace ? true : false }
+      h = {"kind" => "copy_file", "src" => src.to_s, "dst" => dst.to_s, "replace" => replace ? true : false}
       h["mode"] = Integer(mode) unless mode.nil?
       h
     end
@@ -70,13 +70,13 @@ module Microsandbox
     # Copy a host directory (recursively) into the rootfs.
     # @return [Hash]
     def copy_dir(src, dst, replace: false)
-      { "kind" => "copy_dir", "src" => src.to_s, "dst" => dst.to_s, "replace" => replace ? true : false }
+      {"kind" => "copy_dir", "src" => src.to_s, "dst" => dst.to_s, "replace" => replace ? true : false}
     end
 
     # Create a symlink at +link+ pointing to +target+.
     # @return [Hash]
     def symlink(target, link, replace: false)
-      { "kind" => "symlink", "target" => target.to_s, "link" => link.to_s, "replace" => replace ? true : false }
+      {"kind" => "symlink", "target" => target.to_s, "link" => link.to_s, "replace" => replace ? true : false}
     end
 
     # Create a directory (idempotent — no error if it already exists).
@@ -84,7 +84,7 @@ module Microsandbox
     # @param mode [Integer, nil] directory mode (e.g. 0o755)
     # @return [Hash]
     def mkdir(path, mode: nil)
-      h = { "kind" => "mkdir", "path" => path.to_s }
+      h = {"kind" => "mkdir", "path" => path.to_s}
       h["mode"] = Integer(mode) unless mode.nil?
       h
     end
@@ -92,7 +92,7 @@ module Microsandbox
     # Remove a file or directory (idempotent — no error if absent).
     # @return [Hash]
     def remove(path)
-      { "kind" => "remove", "path" => path.to_s }
+      {"kind" => "remove", "path" => path.to_s}
     end
   end
 end