safrano 0.6.6 → 0.6.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '0108c5a48b2cbd994534ba6732f760dec8f8e8becac690d91b3f1960dd6f7836'
-  data.tar.gz: c9b8c3c8a7f7ac3d7db5a12133ba3fdcd4815df5b3dab1d35bfaf449518f8d2c
+  metadata.gz: a93263d81a1f266029f7aac03d6b7226477f10d27d33255462e08133955ab759
+  data.tar.gz: 894a0bfab2eeeee57543bee7dd0c1ec93b5bc73e0afc6fe5f979c7b7e7067039
 SHA512:
-  metadata.gz: '0068c08576a0d8df023d6990ad691ce73b1d3abd6f3a9cd095b7ae2f9358dc7d33d7ed5d0fdde2503a2829ebeb11ab126961a39d34d12370befbb5814904f7b1'
-  data.tar.gz: cbf6fbdd2d1f4a24ba58b14490dfcfc87e1ffc333ab52b0e334b06742679543a794359f16093f56cc78e7a754724ee3e24f70ec7b7277d3ce6d04af3d1f9ae5f
+  metadata.gz: f0900d4f692a9139226d4fb84703f8a0c47a071b97b0a46c8455016f6a7d0d09caf2b12f32e616c9880a10a0b50e09917ca22afc4af7fc0f4cddac690193b25c
+  data.tar.gz: 30a623f667830e0dc4342bcf240843d313173ef12c0e877dc1814e779fc7f53459e7ec7e958aedd22b4387c9390907844a0c2c84a639db8d98dc62805203a671

data/lib/odata/collection_media.rb

@@ -21,7 +21,7 @@ module Safrano
     class Static < Handler
       def initialize(root: nil, mediaklass:)
         @root = File.absolute_path(root || Dir.pwd)
-        @file_server = ::Rack::File.new(@root)
+        @file_server = ::Rack::Files.new(@root)
         @media_class = mediaklass
         @media_dir_name = mediaklass.to_s
         register

data/lib/odata/function_import.rb

@@ -10,6 +10,21 @@ module Safrano
   end
 
   module FunctionImport
+    # error classes
+    class DefinitionMissing < StandardError
+      def initialize(fnam)
+        msg = "Function import #{fnam}: definition is missing. Provide definition either as a return code block or with .definition(lambda)"
+        super(msg)
+      end
+    end
+    class ProcRedefinition < StandardError
+      def initialize(fnam)
+        msg = "Function import #{fnam}: Block/lambda Redefinition . Provide definition either as a return code block or with .definition(lambda) but not both"
+        super(msg)
+      end
+    end
+
+    # Function import object
     class Function
       @allowed_transitions = [Safrano::TransitionEnd]
       attr_reader :name
@@ -18,7 +33,6 @@ module Safrano
       def initialize(name)
         @name = name
         @http_method = 'GET'
-        @use_contract = false
       end
 
       def allowed_transitions
@@ -53,13 +67,7 @@ module Safrano
       end
       alias auto_query_params auto_query_parameters
 
-      def use_contract
-        @use_contract = true
-      end
-
       def return(klassmod, &proc)
-        raise('Please provide a code block') unless block_given?
-
         @returning = if klassmod.respond_to? :return_as_instance_descriptor
                        klassmod.return_as_instance_descriptor
                      else
@@ -67,13 +75,35 @@ module Safrano
                        # --> assume it is a Primitive
                        ResultDefinition.asPrimitiveType(klassmod)
                      end
-        @proc = proc
+        # block is optional  since 0.6.7
+        # the function definition can now also be made with .definition(lambda)
+        # consistency check that there is a single definition either as a
+        # return-block    or a definition lambda is made on publish finalise
+
+        if block_given?
+          # proc already defined...
+          raise Redefinition.new(@name) if @proc
+
+          @proc = proc
+        end
+
         self
       end
 
-      def return_collection(klassmod, &proc)
-        raise('Please provide a code block') unless block_given?
+      def definition(lambda)
+        raise('Please provide a lambda') unless lambda
+        # proc already defined...
+        raise ProcRedefinition.new(@name) if @proc
+
+        @proc = lambda
+      end
+
+      # this is called from service.finalize_publishing
+      def check_definition
+        raise DefinitionMissing.new(@name) unless @proc
+      end
 
+      def return_collection(klassmod, lambda: nil, &proc)
         @returning = if klassmod.respond_to? :return_as_collection_descriptor
                        klassmod.return_as_collection_descriptor
                      else
@@ -82,7 +112,14 @@ module Safrano
                        #                       ResultAsPrimitiveTypeColl.new(klassmod)
                        ResultDefinition.asPrimitiveTypeColl(klassmod)
                      end
-        @proc = proc
+        # block is optional  since 0.6.7
+        if block_given?
+          # proc already defined...
+          raise ProcRedefinition.new(@name) if @proc
+
+          @proc = proc
+        end
+
         self
       end
       #      def initialize_params
@@ -156,8 +193,13 @@ module Safrano
       def with_transition_validated(req)
         #        initialize_params
         @params = req.params
-        return yield unless (@error = check_url_func_params)
-
+        unless (@error = check_url_func_params)
+          begin
+            return yield
+          rescue LocalJumpError => e
+            @error = Safrano::ServiceOperationReturnError.new
+          end
+        end
         [nil, :error, @error] if @error
       end
 

data/lib/odata/model_ext.rb

@@ -753,7 +753,8 @@ module Safrano
           # json is default content type so we dont need to specify it here again
           # TODO quirks array mode !
           # [201, EMPTY_HASH, new_entity.to_odata_post_json(service: req.service)]
-          [201, { 'Location' => new_entity.uri }, new_entity.to_odata_create_json(request: req)]
+          [201, { Safrano::LOCATION => new_entity.uri },
+           new_entity.to_odata_create_json(request: req)]
         else # TODO: other formats
           415
         end

data/lib/odata/request/json.rb

@@ -1,9 +1,15 @@
 require 'json'
 require 'time'
-
+require 'base64'
 # client parsing functionality to ease testing
 
 module Safrano
+  module RFC2047
+    def self.encode(str)
+      "=?utf-8?b?#{Base64.strict_encode64(str)}?="
+    end
+  end
+
   module OData
     # this is used to parse inbound json payload on POST / PUT & co
     # it does not do symbolize but proper (hopefully) type casting when needed

data/lib/safrano/core.rb

@@ -13,10 +13,10 @@ module Safrano
 
   # some prominent constants... probably already defined elsewhere eg in Rack
   # but lets KISS
-  CONTENT_TYPE = 'Content-Type'
-  CONTENT_LENGTH = 'Content-Length'
-  LOCATION = 'Location'
-  CTT_TYPE_LC = 'content-type'
+  CONTENT_TYPE = 'content-type'
+  CONTENT_LENGTH = 'content-length'
+  LOCATION = 'location'
+
   TEXTPLAIN_UTF8 = 'text/plain;charset=utf-8'
   APPJSON = 'application/json'
   APPXML = 'application/xml'

data/lib/safrano/multipart.rb

@@ -142,12 +142,12 @@ module MIME
           @lines = inpstr.readlines(@sep)
         else
           # rack input wrapper only has gets but not readlines
-          sepsave = $INPUT_RECORD_SEPARATOR
-          $INPUT_RECORD_SEPARATOR = @sep
-          while (line = inpstr.gets)
-            @lines << line
-          end
-          $INPUT_RECORD_SEPARATOR = sepsave
+          # BUT the rack SPEC says it only supports gets without argument!
+          # --> finally we end up using read and split into lines...
+          # normally should be ok for $batch POST payloads
+
+          # inpstr.read should be a String
+          @lines = inpstr.read.lines(@sep)
 
         end
         # tmp hack for test-tools that convert CRLF in payload to LF :-(

data/lib/safrano/service.rb

@@ -375,7 +375,7 @@ module Safrano
     def finalize_publishing
       # build the cmap
       @cmap = {}
-      @collections.each do |klass|
+      @collections&.each do |klass|
         @cmap[klass.entity_set_name] = klass
         # set namespace needed to have qualified type name
         copy_namespace_to(klass)
@@ -393,7 +393,7 @@ module Safrano
       set_uribase
 
       # finalize the uri's and include NoMappingBeforeOutput or MappingBeforeOutput as needed
-      @collections.each do |klass|
+      @collections&.each do |klass|
         klass.finalize_publishing(self)
 
         klass.build_uri(@uribase)
@@ -448,9 +448,14 @@ module Safrano
         Safrano::Filter::DateTimeLit.include Safrano::Filter::DateTimeDefault
         Safrano::Filter::DateTimeOffsetLit.include Safrano::Filter::DateTimeDefault
       end
+
+      # check function import definition
+      function_imports.each_value { |func| func.check_definition }
     end
 
     def execute_deferred_iblocks
+      return unless @collections
+
       @collections.each do |k|
         k.instance_eval(&k.deferred_iblock) if k.deferred_iblock
       end
@@ -460,6 +465,8 @@ module Safrano
     ## evaluated after '@collections' is filled !
     # A regexp matching all allowed base entities (eg product|categories )
     def base_url_regexp
+      return unless @collections
+
       @collections.map(&:entity_set_name).join('|')
     end
 

data/lib/safrano/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Safrano
-  VERSION = '0.6.6'
+  VERSION = '0.6.7'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: safrano
 version: !ruby/object:Gem::Version
-  version: 0.6.6
+  version: 0.6.7
 platform: ruby
 authors:
 - oz
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-01-04 00:00:00.000000000 Z
+date: 2023-02-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1.1'
 - !ruby/object:Gem::Dependency
   name: rfc2047
   requirement: !ruby/object:Gem::Requirement
@@ -114,14 +114,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement