faastruby 0.5.5 → 0.5.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8a491f4abb96d8fa8365469a5cd1adec928ba2c775e05dedafbcce10924ef889
-  data.tar.gz: e20f906502710fb4b7bb5803a47e4844d183e254305766e8f5fff94341b5ef63
+  metadata.gz: f6c5b8269b49257d206406eb93f408d375f9c6ebdfe50f0fac40ee62a2fd8db3
+  data.tar.gz: b289988fe2d42f3c49062fb19810c664cf4655a3296c6d5726390bad3c77d67b
 SHA512:
-  metadata.gz: 77f0c2999f8648cad2055da5a36917ece93849b7e243d5d0c512bde0406b2fffeb4a121686b5a420b4a4ed9ed49e3d10af3ada202bda8d2d9528fa3df675b8db
-  data.tar.gz: 22f7921d21da2357857f2b4e66bd0cc0c0334e0c2f610e8c4538abf629874889121e99efc835867a2645a709dce0611784a23568f8f3181ee88e20858698a969
+  metadata.gz: c34f895e38b4a72493e252ff84c6f3129a217fe36a7f68fb26775f06da8b129aa85cb262c1896410aae857720667cb35a3a83cae333077e94489e9d55698ff40
+  data.tar.gz: 4c80da8ed9e3a45bd319d6ad726ebd8ab1f1dd7fb99f25db70adb58806544bbe5336046df3d1b48da8980238abc3a840431e904329d8f855d5b8fe412f9c2deb

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.5.6 - Mar 10 2019
+- Fix bug preventing require libraries. Thanks to
+- Support for static files with space in their names
+- Cleanup comments code
+- Limit the size of static files to 5MB
+- Check if the number of runners is an integer
+
 ## 0.5.5 - Mar 9 2019
 - Use a class method inside FaaStRuby::Response to handle invalid response messages.
 

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    faastruby (0.5.4)
+    faastruby (0.5.5)
       colorize (~> 0.8)
       faastruby-rpc (~> 0.2)
       listen (~> 3.1)

data/lib/faastruby/cli/commands/workspace/cp.rb

@@ -3,6 +3,7 @@ module FaaStRuby
     module Workspace
       require 'tmpdir'
       require 'tempfile'
+      require 'uri'
       # require 'faastruby/cli/commands/workspace/base_command'
       require 'faastruby/cli/package'
       require 'faastruby/cli/new_credentials'
@@ -20,8 +21,9 @@ module FaaStRuby
         end
 
         def run
-          destination_url = "#{FaaStRuby.workspace_host_for(@workspace_name)}/#{@relative_path}"
-          spinner = say("[#{@source_file}] Copying file to '#{destination_url}'...", quiet: true)
+          FaaStRuby::CLI.error("You can't upload static files larger than 5MB. If you need to upload large files, please reach out to us on Slack at https://faastruby.io/slack") if source_file_too_big?
+          destination_url = URI.escape("#{FaaStRuby.workspace_host_for(@workspace_name)}/#{@relative_path}")
+          # spinner = say("[#{@source_file}] Copying file to '#{destination_url}'...", quiet: true)
           workspace = FaaStRuby::Workspace.new(name: @workspace_name)
           package = build_package
           workspace.upload_file(package, relative_path: @relative_path)
@@ -34,12 +36,14 @@ module FaaStRuby
           @package_file.unlink
         end
 
+        def source_file_too_big?
+          File.size(@source_file) > 5242880 # That's 5MB
+        end
 
         def build_package
           FileUtils.cp @source_file, @tmpdir
           output_file = @package_file.path
           FaaStRuby::Package.new(@tmpdir, output_file).build
-          # FaaStRuby::Command::Function::Build.build(@source_file, output_file, @function_name, true)
           @package_file.close
           output_file
         end
@@ -77,30 +81,6 @@ module FaaStRuby
           FaaStRuby::CLI.error("You can only 'cp' files, not directories.") unless File.file?(@source_file)
           true
         end
-
-        # def missing_args
-        #   FaaStRuby::CLI.error(["'#{@workspace_name}' is not a valid workspace name.".red, usage], color: nil) if @workspace_name =~ /^-.*/
-        #   @missing_args << "Missing argument: WORKSPACE_NAME".red unless @workspace_name
-        #   @missing_args << "Missing argument: -s SOURCE_FILE" unless @options['source']
-        #   @missing_args << "Missing argument: -d DESTINATION_PATH" unless @options['destination']
-        #   @missing_args << usage if @missing_args.any?
-        #   @missing_args
-        # end
-
-        # def parse_options(require_options: {})
-        #   @options = {}
-        #   while @args.any?
-        #     option = @args.shift
-        #     case option
-        #     when '-s', '--source'
-        #       @options['source'] = @args.shift
-        #     when '-d', '--destination'
-        #       @options['destination'] = @args.shift.gsub(/(^\/|\/$|\.\.|;|'|"|&|\\)/, '')
-        #     else
-        #       FaaStRuby::CLI.error("Unknown argument: #{option}")
-        #     end
-        #   end
-        # end
       end
     end
   end

data/lib/faastruby/cli/commands/workspace/update.rb

@@ -51,6 +51,7 @@ module FaaStRuby
             case option
             when '--runners'
               @options['runners_max'] = @args.shift
+              FaaStRuby::CLI.error("The argument '--runners' takes an integer") if @options['runners_max'].nil? || !@options['runners_max'].to_i.is_a?(Integer)
             else
               FaaStRuby::CLI.error("Unknown argument: #{option}")
             end

data/lib/faastruby/local/static_files/static_file.rb

@@ -4,7 +4,7 @@ module FaaStRuby
       include Local::Logger
 
       def self.full_sync
-        cmd = "faastruby deploy-to #{Local.workspace} -f #{Local.public_dir}"
+        cmd = "faastruby deploy-to '#{Local.workspace}' -f '#{Local.public_dir}'"
         output, status = Open3.capture2e(cmd)
         String.disable_colorization = true
         if status.exitstatus == 0
@@ -27,7 +27,7 @@ module FaaStRuby
 
       def deploy
         path = "public/#{@relative_path}"
-        cmd = "faastruby cp #{path} #{Local.workspace}:/#{@relative_path}"
+        cmd = "faastruby cp '#{path}' '#{Local.workspace}:/#{@relative_path}'"
         puts "Running: #{cmd}"
         output, status = Open3.capture2e(cmd)
         String.disable_colorization = true
@@ -42,7 +42,7 @@ module FaaStRuby
 
       def remove_from_workspace
         path = "public/#{@relative_path}"
-        cmd = "faastruby rm #{Local.workspace}:/#{@relative_path}"
+        cmd = "faastruby rm '#{Local.workspace}:/#{@relative_path}'"
         puts "Running: #{cmd}"
         output, status = Open3.capture2e(cmd)
         String.disable_colorization = true

data/lib/faastruby/server/runner.rb

@@ -19,11 +19,18 @@ module FaaStRuby
 
     def load_function(path)
       eval %(
-        Module.new do
-          def self.require(path)
-            return load("\#{path}.rb") if File.file?("\#{path}.rb")
-            Kernel.require path
+        module Kernel
+          # make an alias of the original require
+          alias_method :original_require, :require
+
+          # rewrite require
+          def require name
+            return load("\#{name}.rb") if File.file?("\#{name}.rb")
+            original_require name
           end
+        end
+
+        Module.new do
           #{File.read(path)}
         end
       )

data/lib/faastruby/version.rb

@@ -1,3 +1,3 @@
 module FaaStRuby
-  VERSION = '0.5.5'
+  VERSION = '0.5.6'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: faastruby
 version: !ruby/object:Gem::Version
-  version: 0.5.5
+  version: 0.5.6
 platform: ruby
 authors:
 - Paulo Arruda
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-03-09 00:00:00.000000000 Z
+date: 2019-03-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rest-client