eco-helpers 2.0.56 → 2.0.59

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c9227a13716393bdd79a7bdf3d4729dbc2b426f43e16d1525001ffd0a2c58bb4
-  data.tar.gz: 00adb8c5c5abc08e53e4130a1cdf9ff03cf4307ac86c4cbe8e84ebffd8ba2458
+  metadata.gz: aec1ba6cc48e340c576f141bec3e128afcb0fe7e072051c8a6da5a6e0fb9c3b1
+  data.tar.gz: a592079366217bd0ec34fbae8a659ed971cc79bdac762a26f5f36e2346f0f944
 SHA512:
-  metadata.gz: 0c69583967b50677bf012fdd354d3fb17abf521fcceb702f71ccfbfd942fb1e11963ff50b103f351e9cbebcfd531751dc9c54094e6b645c40b0e77d5ef6c7d8b
-  data.tar.gz: 60a0e1e311f8a3aae34bbfcf1f4c5dccf3bd541dfaef0ab60b3f7b557a69bc6b5f7a74febdea0a85aa4db658c508da69fef99753c89459db0eab58112c577bc2
+  metadata.gz: e612ff67ebd1148cffc40c064bdcb6e61d119c021b5b1261210699d1f1114bf5d42479822a54545f67947b6f0eee6293607a61947d5e245195cd7a60e1c5d56a
+  data.tar.gz: 2ff3fd08e0f4780a9f0891081357b1e3723ef341c8d038e2f3b3fa508b59349a21ea4bc30f7b71633cdabd73f8d7bd1032b189c67efd7c1316ae64b955d69e25

data/CHANGELOG.md

@@ -1,7 +1,52 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
-## [2.0.56] - 2022-04-xx
+## [2.0.59] - 2022-06-xx
+
+### Added
+  - `Eco::API::Organization::TagTree#leafs` new method to spot tags with no children.
+
+### Changed
+  - `Eco::API::MicroCases#core_excluded`
+    - when person is `new?`, you should still be able to exclude `filter_tags`
+
+### Fixed
+  - `Eco::API::Policies::DefaultPolicies::UserAccess`
+    - `-exclude-account` should leave it untouched
+    
+## [2.0.58] - 2022-05-31
+
+### Added
+  - **Usecase**`Eco::API::UseCases::DefaultCases::ClearAbilitiesTransCase`
+    - invokable with `clear-abilities`
+  - `Eco::API::Session::Config::People` added
+    - `default_login_method` helpers
+  - `Eco::API::Session::Config#default_login_method=` added
+  - `Eco::API::Common::Loaders::Base#abort` method to log an error and raise an exception
+  - **patched** `Ecoportal::API::Internal::Person`
+    - Added a couple of handy helpers: `#replace_doc` and `#replace_original_doc`
+
+### Changed
+  - `Eco::API::Common::Session::SFTP#move` new parameter `override:` (default: `true`)
+    - If there was an error, it would delete the destination remote file and retry to move the file
+    - upgraded `ecoportal-api-v2` gem to `0.8.28`
+  - `Eco::API::Organization::LoginProviders#login_provider`
+    - Added support to fetch based on type (i.e. password, sso)
+  - `Eco::API::Policies::DefaultPolicies::UserAccess`
+    - Added support for `config.default_login_method`
+
+### Fixed
+  - `Eco::API::Common::People::EntryFactory`
+    - Max allowed enconding errors was not working (it was just fixing all).
+
+## [2.0.57] - 2022-04-29
+
+### Changed
+  - `Eco::API::UseCases::DefaultCases::HrisCase`
+    - Made it so the `leavers` batch runs first
+    - This aims to prevent some errors such as `cyclic supervisor` or `email already taken`
+
+## [2.0.56] - 2022-04-25
 
 ### Added
   - `Eco::API::UseCases::OozeSamples::OozeBaseCase#add_field_by_doc`
@@ -31,7 +76,6 @@ All notable changes to this project will be documented in this file.
 ### Added
   - `Eco::API::UseCases::DefaultCases::Samples::Sftp` should tell where the files went in the local.
 
-
 ## [2.0.53] - 2022-04-04
 
 ### Added

data/eco-helpers.gemspec

@@ -31,7 +31,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "redcarpet", ">= 3.5.1",  "< 3.6"
 
   spec.add_dependency 'ecoportal-api', '>= 0.8.5', '< 0.9'
-  spec.add_dependency 'ecoportal-api-v2', '>= 0.8.27', '< 0.9'
+  spec.add_dependency 'ecoportal-api-v2', '>= 0.8.28', '< 0.9'
   spec.add_dependency 'aws-sdk-s3',    '>= 1.83.0', '< 2'
   spec.add_dependency 'aws-sdk-ses',   '>= 1.36.0', '< 2'
   spec.add_dependency 'dotenv',        '>= 2.7.6', '< 2.8'

data/lib/eco/api/common/loaders/base.rb

@@ -51,6 +51,14 @@ module Eco
             session.micro
           end
 
+          def abort(msg, raising: true)
+            logger.error(msg)
+            if raising
+              raise msg
+            else
+              exit(1)
+            end
+          end
         end
       end
     end

data/lib/eco/api/common/people/default_parsers/boolean_parser.rb

@@ -11,7 +11,7 @@ class Eco::API::Common::People::DefaultParsers::BooleanParser < Eco::API::Common
 
   private
 
-  def truthy (value)
+  def truthy(value)
     %w[true y yes x].include?(value.to_s.strip.downcase)
   end
 

data/lib/eco/api/common/people/entry_factory.rb

@@ -208,6 +208,7 @@ module Eco
                   logger.error("There were more than 5 encoding errors in the file '#{file}'.")
                   return content
                 else
+                  tolerance -= 1
                   logger.error("Encoding problem in file '#{file}': '#{replacement}'.")
                   replacement
                 end

data/lib/eco/api/common/session/sftp.rb

@@ -56,8 +56,14 @@ module Eco
           end
 
           # @see Net::SFTP::Session#rename
-          def move(fullname_source, fullname_dest, flags=0x0001, &callback)
-            sftp_session.rename!(fullname_source, fullname_dest, flags, &callback)
+          def move(fullname_source, fullname_dest, flags=0x0001, override: true, &callback)
+            begin
+              sftp_session.rename!(fullname_source, fullname_dest, flags, &callback)
+            rescue Net::SFTP::StatusException => e
+              raise unless override
+              sftp_session.remove(fullname_dest)
+              sftp_session.rename!(fullname_source, fullname_dest, flags, &callback)
+            end
           end
 
           # Downloads the files specified to a local folder

data/lib/eco/api/common/version_patches/ecoportal_api/internal_person.rb

@@ -14,6 +14,23 @@ module Ecoportal
           account && !ref_doc["account"]
         end
 
+        # It replaces `doc` by `new_doc`
+        # @return [Hash] `doc` before change
+        def replace_doc!(new_doc)
+          raise UnlinkedModel.new(from: "#{self.class}#replace_doc", key: _key) unless linked?
+          @doc.tap do
+            @doc = new_doc
+          end
+        end
+
+        # It replaces `original_doc` by `new_doc`
+        # @return [Hash] `original_doc` before change
+        def replace_original_doc!(new_doc)
+          raise UnlinkedModel.new(from: "#{self.class}#replace_original_doc", key: _key) unless linked?
+          @original_doc.tap do
+            @original_doc = new_doc
+          end
+        end
       end
     end
   end

data/lib/eco/api/microcases/core_excluded.rb

@@ -7,12 +7,11 @@ module Eco
       # @return [Array<String>] the core parameters that should not be included.
       def core_excluded(person, options)
         ["supervisor_id"].tap do |core_excluded|
-          unless person.new?
-            exclusions = ["name", "external_id", "email", "filter_tags"].select do |attr|
-              options.dig(:exclude, attr.to_sym)
-            end
-            core_excluded.concat(exclusions)
+          can_exclude = person.new?? ["filter_tags"] : ["name", "external_id", "email", "filter_tags"]
+          exclusions  = can_exclude.select do |attr|
+            options.dig(:exclude, attr.to_sym)
           end
+          core_excluded.concat(exclusions)
         end
       end
     end

data/lib/eco/api/microcases/set_supervisor.rb

@@ -3,7 +3,7 @@ module Eco
     class MicroCases
       # Unique access point to set the `supervisor_id` value on a person.
       # @note
-      #   - It prevents the basic cyclic supervisor case (supervisor to be supervisor of hemselves)
+      #   - It prevents the basic cyclic supervisor case (supervisor to be supervisor of themselves)
       # @param person [Ecoportal::API::V1::Person] the person we want to update, carrying the changes to be done.
       # @param sup_id [nil, String] the **supervisor id** we should set on the `person`.
       # @param people [Eco::API::Organization::People] _People_ involved in the current update.

data/lib/eco/api/organization/login_providers.rb

@@ -56,27 +56,41 @@ module Eco
           end
         end
 
-        def login_provider(id_name)
-          @by_id.fetch(login_provider_id(id_name), nil)
+        # @param id_name_type [String, Symbol]
+        #   - `String` to refer to the login provider `name` or `id`
+        #   - `Symbol` to refer to the login providr type (i.e. `:password`, `:sso`)
+        def login_provider(id_name_type)
+          @by_id[login_provider_id(id_name_type)]
         end
 
         private
 
-        def login_provider_name(id_name)
-          (@by_id[id_name] || @by_name[id_name&.downcase])&.name&.downcase
-        end
-
-        def login_provider_id(id_name)
-          (@by_name[id_name&.downcase] || @by_id[id_name])&.id
+        def login_provider_id(value)
+          case value
+          when Symbol
+            value = value.to_s.downcase.to_sym
+            @by_type[value]&.id
+          when String
+            value = value.strip.downcase
+            @by_name[value]&.id || @by_id[value]&.id
+          end
         end
 
         def init_caches
           return if @caches_init
-          @by_id   = self.map { |pg| [pg.id, pg] }.to_h
-          @by_name = self.map { |pg| [pg.name&.downcase, pg] }.to_h
+          enabled_lps = self.select {|lp| lp.enabled_for != "disabled"}
+          @by_id   = enabled_lps.each_with_object({}) do |lp, hash|
+            hash[lp.id.downcase] = lp
+          end
+          @by_name = enabled_lps.each_with_object({}) do |lp, hash|
+            hash[lp.name.downcase] = lp
+          end
+          @by_type = enabled_lps.each_with_object({}) do |lp, hash|
+            hash[lp.type.downcase.to_sym] = lp
+            hash[:sso] = lp if (!hash.key?(:sso) && lp.type == "saml")
+          end
           @caches_init = true
         end
-
       end
     end
   end

data/lib/eco/api/organization/tag_tree.rb

@@ -124,6 +124,14 @@ module Eco
           subtags.include?(key&.upcase)
         end
 
+        # Returns all the tags with no children
+        # @return [Array<String>]
+        def leafs
+          tags.select do |tag|
+            node(tag).children_count == 0
+          end
+        end
+
         # Verifies if a tag exists in the tree.
         # @param key [String] tag to verify.
         # @return [Boolean]

data/lib/eco/api/policies/default_policies/99_user_access_policy.rb

@@ -1,3 +1,7 @@
+# Default policy:
+# 1. Removes the account if the email is empty
+# 2. Adds the defined default_usergroup if user does not have policy groups
+# 3. Adds the defined default_login_method if update includes empty login methods
 class Eco::API::Policies::DefaultPolicies::UserAccess < Eco::API::Common::Loaders::Policy
   name "default-user-access"
 
@@ -10,8 +14,13 @@ class Eco::API::Policies::DefaultPolicies::UserAccess < Eco::API::Common::Loader
     self.account_removed_count = 0
 
     people.each do |person|
-      remove_account_when_no_email!(person)    if person.email.to_s.empty?
-      person.account.policy_group_ids = defid  if no_policy_group_ids?(person)
+      remove_account_when_no_email!(person) if person.email.to_s.empty?
+      next unless account = person.account
+      next if options.dig(:exclude, :account)
+      if account.policy_group_ids.empty? && def_pg
+        account.policy_group_ids = [def_pg]
+      end
+      add_login_method_if_applicable!(account)
     end
 
     warn_account_removal!
@@ -39,20 +48,34 @@ class Eco::API::Policies::DefaultPolicies::UserAccess < Eco::API::Common::Loader
     return !!person.original_doc["account"]
   end
 
-  def no_policy_group_ids?(person)
-    (account = person.account) &&  account.policy_group_ids.empty?
+  def add_login_method_if_applicable!(account)
+    return unless account.as_update.key?("login_provider_ids")
+    if account.login_provider_ids.empty? && def_login
+      account.login_provider_ids = [def_login]
+    end
+  end
+
+  def def_pg
+    @def_pg ||= policy_groups.to_id(default_group)
   end
 
-  def defid
-    @defid ||= policy_groups.to_id([default_group]).compact
+  def def_login
+    @def_login ||= login_providers.to_id(default_login)
   end
 
   def default_group
-    session.config.people.default_usergroup
+    config.people.default_usergroup
+  end
+
+  def default_login
+    config.people.default_login_method
+  end
+
+  def login_providers
+    session.login_providers
   end
 
   def policy_groups
     session.policy_groups
   end
-
 end

data/lib/eco/api/session/config/people.rb

@@ -86,7 +86,19 @@ module Eco
           def default_schema?
             !!self["default_schema"]
           end
-          
+
+          def default_login_method=(name)
+            self["default_login_method"] = name
+          end
+
+          def default_login_method
+            self["default_login_method"]
+          end
+
+          def default_login_method?
+            !!self["default_login_method"]
+          end
+
           # @return [Hash] with defined pairs format `key` and Person parsers.
           def parsers
             self["parsers"] ||= {}

data/lib/eco/api/session/config.rb

@@ -297,6 +297,11 @@ module Eco
           people.default_schema = name
         end
 
+        # Set the **default login method** that users should acquire.
+        def default_login_method=(name)
+          people.default_login_method = name
+        end
+
         # @see Eco::API::Session::Config::People
         # @param (see Eco::API::Session::Config::People)
         # @return [Eco::API::Common::People::PersonParser] parser/serializer for the defined `format`.

data/lib/eco/api/usecases/default_cases/clear_abilities_case.rb

@@ -0,0 +1,22 @@
+class Eco::API::UseCases::DefaultCases::ClearAbilitiesTransCase < Eco::API::Common::Loaders::UseCase
+  name "clear-abilities"
+  type :transform
+
+  def main(people, session, options, usecase)
+    clear_abilities(people.users, session, options, usecase)
+  end
+
+  private
+
+  def clear_abilities(users, session, options, usecase)
+    update   = session.new_job("main", "clear-abilities", :update, usecase, :account)
+    users.each do |person|
+      person.account.permissions_custom = nil_abilities.dup
+      update.add(person)
+    end
+  end
+
+  def nil_abilities
+    @nil_abilities ||= Eco::API::Organization::PresetsFactory.all_abilities
+  end
+end

data/lib/eco/api/usecases/default_cases/hris_case.rb

@@ -9,10 +9,10 @@ class Eco::API::UseCases::DefaultCases::HrisCase < Eco::API::Common::Loaders::Us
     @session = session; @options = options; @people = people
     require_only_one_schema!
     micro    = session.micro
+    @leavers  = session.new_job("pre", "leavers",  :update, usecase, :account)
     @creation = session.new_job("main", "create",  :create, usecase)
     @update   = session.new_job("main", "update",  :update, usecase)
     @supers   = session.new_job("post", "supers",  :update, usecase, :core)
-    @leavers  = session.new_job("post", "leavers", :update, usecase, :account)
 
     micro.with_each_leaver(entries, people, options) do |person|
       leavers.add(person, &method(:leavers_callback))

data/lib/eco/api/usecases/default_cases.rb

@@ -15,6 +15,7 @@ require_relative 'default_cases/analyse_people_case'
 require_relative 'default_cases/append_usergroups_case'
 require_relative 'default_cases/change_email_case'
 require_relative 'default_cases/clean_unknown_tags_case'
+require_relative 'default_cases/clear_abilities_case'
 require_relative 'default_cases/codes_to_tags_case'
 require_relative 'default_cases/create_case'
 require_relative 'default_cases/create_details_case'

data/lib/eco/cli/config/default/usecases.rb

@@ -196,6 +196,10 @@ ASSETS.cli.config do |cnf|
       options.deep_merge!(force: {invite: true})
     end
 
+    desc  = "Clears the `permissions_custom`, provided that Policy Group abilities alone apply."
+    desc += " Please be mindfull if you use this option (there may be users that have been upgraded in ad-hoc manner)."
+    cases.add("-clear-abilities", :transform, desc, case_name: "clear-abilities")
+
     desc = "Deletes everybody that has been filtered from the people manager"
     cases.add("-delete", :transform, desc, case_name: "delete")
     desc = "Deletes the people specified in the input data"

data/lib/eco/version.rb

@@ -1,3 +1,3 @@
 module Eco
-  VERSION = "2.0.56"
+  VERSION = "2.0.59"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: eco-helpers
 version: !ruby/object:Gem::Version
-  version: 2.0.56
+  version: 2.0.59
 platform: ruby
 authors:
 - Oscar Segura
@@ -136,7 +136,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.27
+        version: 0.8.28
     - - "<"
       - !ruby/object:Gem::Version
         version: '0.9'
@@ -146,7 +146,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.27
+        version: 0.8.28
     - - "<"
       - !ruby/object:Gem::Version
         version: '0.9'
@@ -528,6 +528,7 @@ files:
 - lib/eco/api/usecases/default_cases/append_usergroups_case.rb
 - lib/eco/api/usecases/default_cases/change_email_case.rb
 - lib/eco/api/usecases/default_cases/clean_unknown_tags_case.rb
+- lib/eco/api/usecases/default_cases/clear_abilities_case.rb
 - lib/eco/api/usecases/default_cases/codes_to_tags_case.rb
 - lib/eco/api/usecases/default_cases/create_case.rb
 - lib/eco/api/usecases/default_cases/create_details_case.rb