model_driven_api 2.3.7 → 2.3.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ec88dcaf27ab5abca89f8db87d1293237950dc1879565048b4fbb3d2901b7ac3
-  data.tar.gz: 643513f277c72318ac7b3ffb9ef2e165258113f093611e29442e24a0aa5d553d
+  metadata.gz: 96a404941f09e02fd096c7b7f62cd9ba80ee3a61058652e5b74961167d7d123c
+  data.tar.gz: 821c7372263973d516017ce7704a51a851cbb87a2259b9540c548b18aa506f30
 SHA512:
-  metadata.gz: 76eccc737ca72f10b10b7b0ae265560f41c53950c423faa8c78fece2097e502c1fe477809720e17acff7bdf796d1a5d186d8710d7fe099d3da6e12f987a2fb88
-  data.tar.gz: 4a9052111ab66b72e8639fa2d567bb4cfd99b565fa304fc17e2e80d8eb1d3627a526f3a2f7fcc8cc5229f4816bb578085cc06a6eef85028f603fb10538e51699
+  metadata.gz: e9ce5471c94e2de11172bafcd047ce527840eb950eaef9281d9030542bcdc3ac7149edd81b366f38e6b8c7abe2214b4169f89c1412134d4f89e68a8f977db93c
+  data.tar.gz: 68756bc2801c58b653e12cbd6b02a0cb5379b5c6471246f371d56f8cae0c40eb41172b1c410819c2562507d80d2e8a76c03b56e7e8dc0f7d8f519bebd329fced

data/README.md

@@ -1,8 +1,9 @@
 # Model Driven Api
 
-## Goal
+## Goals
 
-To have a comprehensive and meaningful Model driven API right out of the box by just creating migrations in your rails app or engine. With all the CRUD operations in place out of the box and easily expandable with custom actions if needed.
+* To have a comprehensive and meaningful Model driven API right out of the box by just creating migrations in your rails app or engine. With all the CRUD operations in place out of the box and easily expandable with custom actions if needed.
+* To have a plain REST implementation which adapts the returned JSON to the specific needs of the client, **without the need to change backend code**, this may overcome the biggest disadvantage of REST vs GraphQL = client driver presentation.
 
 ## TL;DR 5-10 minutes adoption
 
@@ -27,6 +28,55 @@ I've always been interested in effortless, no-fuss, conventions' based developme
 
 Doing this means also narrowing a bit the scope of the tools, taking decisions, at least for the first implementations and versions of this engine, so, this works well if the data is relational, this is a prerequisite (postgres, mysql, mssql, etc.).
 
+## REST Enhanced
+
+Thanks to the inclusion of [Ransack](https://github.com/activerecord-hackery/ransack/wiki) and [ActiveModel::Serializer](https://api.rubyonrails.org/classes/ActiveModel/Serializers/JSON.html), by just adding the querystring keys **q** and **a**, you can create complex queries (q) to obtain just the records you need, which present in the returnd JSON just the attributes (a) needed.
+By combining the two keys, you can obtain just the data you want.
+
+The *json_attrs* or *a* query string passed accepts these keys (Please see [ActiveModel::Serializer](https://api.rubyonrails.org/classes/ActiveModel/Serializers/JSON.html) for reference):
+- only: list [] of model fields to be shown in JSON serialization
+- except: exclude these fields from the JSON serialization, this is a list []
+- methods: include the result of some methods defined in the model, this is a list []
+- include: include associated models, it's an object {} which also accepts the keys described in this document (only, except, methods, include)
+
+### Example
+
+```
+{{ base_url  }}/{{ controller  }}?a[only][]=locked&a[only][]=username&a[methods][]=jwe_data
+```
+
+Is translated to:
+
+```
+{a: {only: ["locked", "username"], methods: ["jwe_data"]}}
+```
+
+Which tells the API controller just to return this optimized serialization:
+
+```
+[
+  {
+    "username": "Administrator",
+    "locked": false,
+    "jwe_data": "eyJhbGciOiJkaXIiLCJlbmMiOiJSMTI4R0NNIn0..yz0tnC6y3BzgoOsO.BjHb9CRIb0vrv7nnEx54Ac8-cATPJ9sTlQSSxRbTmtcPHc5KhvtyE_hyBRnIcK92bzUBRwdy6ASB2XJVy1VfWxAmO8E.4tOzJlfuXi-shaRhDSkOyg"
+  }
+]
+```
+
+By combinig with Ransack's **q** query string key (please read [Ransack](https://github.com/activerecord-hackery/ransack/wiki) documentation to discover all the possible and complex searches you can make), you can obtain right what you want:
+
+```
+{{ base_url  }}/{{ controller  }}?a[only][]=locked&a[only][]=username&a[methods][]=jwe_data&q[email_cont][]=adm
+```
+
+Which translates to:
+
+```
+{a: {only: ["locked", "username"], methods: ["jwe_data"]}, q: { email_cont: ["adm"]}}
+```
+
+For bigger searches, which may over crowd the querystring length, you can always use the default [Search](#Search) POST endpoint.
+
 ## v2?
 
 Yes, this is the second version of such an effort and you can note it from the api calls, which are all under the ```/api/v2``` namespace the [/api/v1](https://github.com/gabrieletassoni/thecore_api) one, was were it all started, many ideas are ported from there, such as the generation of the automatic model based crud actions, as well as custom actions definitions and all the things that make also this gem useful for my daily job were already in place, but it was too coupled with [thecore](https://github.com/gabrieletassoni/thecore)'s [rails_admin](https://github.com/sferik/rails_admin) UI, making it impossible to create a complete UI-less, API only application, out of the box and directly based of the DB schema, with all the bells and whistles I needed (mainly self adapting, data and schema driven API functionalities).

data/app/controllers/api/v2/application_controller.rb

@@ -20,7 +20,10 @@ class Api::V2::ApplicationController < ActionController::API
         return render json: result, status: 200 if status == true
 
         # Normal Index Action with Ransack querying
-        @q = (@model.column_names.include?("user_id") ? @model.where(user_id: current_user.id) : @model).ransack(@query.presence|| params[:q])
+        # Keeping this automation can be too dangerous and lead to unpredicted results
+        # TODO: Remove it
+        # @q = (@model.column_names.include?("user_id") ? @model.where(user_id: current_user.id) : @model).ransack(@query.presence|| params[:q])
+        @q = @model.ransack(@query.presence|| params[:q])
         @records_all = @q.result # (distinct: true) Removing, but I'm not sure, with it I cannot sort in postgres for associated records (throws an exception on misuse of sort with distinct)
         page = (@page.presence || params[:page])
         per = (@per.presence || params[:per])
@@ -66,7 +69,9 @@ class Api::V2::ApplicationController < ActionController::API
         return render json: result, status: 200 if status == true
 
         # Normal Create Action
-        @record.user_id = current_user.id if @model.column_names.include? "user_id"
+        # Keeping this automation can be too dangerous and lead to unpredicted results
+        # TODO: Remove it
+        # @record.user_id = current_user.id if @model.column_names.include? "user_id"
         @record.save!
         render json: @record.to_json(json_attrs), status: 201
     end
@@ -108,7 +113,8 @@ class Api::V2::ApplicationController < ActionController::API
             resource = "custom_action_#{params[:do]}"
             raise NoMethodError unless @model.respond_to?(resource)
             # return true, MultiJson.dump(params[:id].blank? ? @model.send(resource, params) : @model.send(resource, params[:id].to_i, params))
-            return true, MultiJson.dump(@model.send(resource, params))
+            puts json_attrs
+            return true, @model.send(resource, params).to_json(json_attrs)
         end
         # if it's here there is no custom action in the request querystring
         return false
@@ -122,18 +128,20 @@ class Api::V2::ApplicationController < ActionController::API
     end
     
     def authenticate_request
-        # puts request.headers.inspect
+        Rails.logger.info request.headers.inspect
         @current_user = nil
-        # puts "Are there wbehooks headers to check for? #{Settings.ns(:security).allowed_authorization_headers}"
+        Rails.logger.info "Are there webhooks headers to check for? #{Settings.ns(:security).allowed_authorization_headers}"
         Settings.ns(:security).allowed_authorization_headers.split(",").each do |header|
             # puts "Found header #{header}: #{request.headers[header.underscore.dasherize]}" 
             check_authorization("Authorize#{header}".constantize.call(request.headers, request.raw_post)) if request.headers[header.underscore.dasherize]
         end
         
-        # This is the default one, if the header doesn't have a valid form for one of the other Auth methods, then use this Auth Class
+        Rails.logger.info "This is the default one, if the header doesn't have a valid form for one of the other Auth methods, then use this Auth Class"
         check_authorization AuthorizeApiRequest.call(request.headers) unless @current_user
+        Rails.logger.info "Inspect @current_user: #{@current_user} if nil, then returns unauthenticated"
         return unauthenticated!(OpenStruct.new({message: @auth_errors})) unless @current_user
         
+        Rails.logger.info "We are here, so the user authenticated"
         current_user = @current_user
         params[:current_user_id] = @current_user.id
         # Now every time the user fires off a successful GET request, 
@@ -143,12 +151,19 @@ class Api::V2::ApplicationController < ActionController::API
     
     def find_record
         record_id ||= (params[:path].split("/").second.to_i rescue nil)
-        @record = @model.column_names.include?("user_id") ? @model.where(id: (record_id.presence || @record_id.presence || params[:id]), user_id: current_user.id).first : @model.find((@record_id.presence || params[:id]))
+        # Keeping this automation can be too dangerous and lead to unpredicted results
+        # TODO: Remove it
+        # @record = @model.column_names.include?("user_id") ? @model.where(id: (record_id.presence || @record_id.presence || params[:id]), user_id: current_user.id).first : @model.find((@record_id.presence || params[:id]))
+        @record = @model.find((@record_id.presence || params[:id]))
         return not_found! if @record.blank?
     end
     
     def json_attrs
-        ((@model.json_attrs.presence || @json_attrs.presence || {}) rescue {})
+        # In order of importance: if you send the configuration via querystring you are ok 
+        # has precedence over if you have setup the json_attrs in the model concern
+        from_params = params[:a].deep_symbolize_keys unless params[:a].blank? 
+        from_params = params[:json_attrs].deep_symbolize_keys unless params[:json_attrs].blank?
+        from_params.presence || @json_attrs.presence || @model.json_attrs.presence || {} rescue {}
     end
     
     def extract_model

data/config/initializers/time_with_zone.rb

@@ -0,0 +1,8 @@
+# Turns API calls to always UTC, leaving the APP the freedom to use local timezone
+module ActiveSupport
+    class TimeWithZone
+      def as_json(options = nil)
+        utc
+      end
+    end
+end

data/lib/concerns/api_exception_management.rb

@@ -10,6 +10,7 @@ module ApiExceptionManagement
             rescue_from ActiveModel::ForbiddenAttributesError, with: :fivehundred!
             rescue_from ActiveRecord::RecordInvalid, with: :invalid!
             rescue_from ActiveRecord::RecordNotFound, with: :not_found!
+            rescue_from ActiveRecord::RecordNotUnique, with: :invalid!
         end
         
         def unauthenticated! exception = AuthenticateUser::AccessDenied.new

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: model_driven_api
 version: !ruby/object:Gem::Version
-  version: 2.3.7
+  version: 2.3.12
 platform: ruby
 authors:
 - Gabriele Tassoni
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-02-26 00:00:00.000000000 Z
+date: 2021-04-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thecore_backend_commons
@@ -142,6 +142,7 @@ files:
 - config/initializers/after_initialize_for_model_driven_api.rb
 - config/initializers/cors_api_thecore.rb
 - config/initializers/knock.rb
+- config/initializers/time_with_zone.rb
 - config/initializers/wrap_parameters.rb
 - config/routes.rb
 - lib/concerns/api_exception_management.rb