arca.rb 1.0.1 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c8d2177ff2a7410f72cf2b1cf7e730ed4b683a49980cf26d6dd861e6d08e1025
-  data.tar.gz: 066b1a6a6ae7b089cc95a91d3ce034ddec3911d7cbc3c43fc16168a0e2bb85e5
+  metadata.gz: dd4d89db0b9d4d77abb2b5e6eb0e03d3169893d511a0e2b066f12e3c1d613e2c
+  data.tar.gz: 49b8a96793606351fa4a9ca0b137cab1f7efc2459ef5604d9efffe0a20c5b54f
 SHA512:
-  metadata.gz: e2da9b334aa535351083be80a9734cd65777947990ccdbc48b369b71a97e9fc3bb5b6ea56e767493f9147c218159ae3bce47bcc713c469163b6d1033947fc1d1
-  data.tar.gz: 8f8a371d97b9ee91b03d345b1f38973668e4e81b7294f7450d4a18e5f09306462ef9831b8a1d2dae7759edec5d0294f8c83b27512aba5077c827b4b9c61b021a
+  metadata.gz: 4b6060fe8e15d82e5ca2cf8cfa5f7651899ae10661df16a389b6cb43ddcfbed79f6303b17a126b654d40e1b5830ba78263f6a20f7af6d11a1fe36f89eee8c459
+  data.tar.gz: 02242344d53082a0b5b7f3b6aba1f004184b8aa8ac64e91ac9ed07b7f623a95191fe82582f0acd14256d616d5b2928f59b07471c73351887dcf7fb4b8af7342f

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Arca Changelog
 
+## [1.1.0]
+
+- **WSAA:** Store opcional para TA. Podés pasar `store:` con un objeto que implemente `read(cuit:, env:, service:)` y `write(cuit:, env:, service:, ta:, expires_at:)` para guardar el token en tu backend (p. ej. DB por tenant con cifrado). Sin `store`, se sigue usando el archivo JSON en `tmp/`.
+
+## [1.0.2] - 2025-02-09
+
+- **Errores:** En WSAA, `persist_ta` ahora encapsula fallos en `Arca::ServerError` en lugar de re-levantar excepciones crudas (mejor encapsulación y menos riesgo de filtrar datos sensibles en logs).
+- **Test:** Añadido test que verifica que `persist_ta` levanta `ServerError` con la excepción original en `cause`.
+- Actualización de gems/dependencias.
+
 ## [1.0.1] - 2025-02-05
 - Fix ruta hardcodeada de certificados SSL CA. Usar OpenSSL::X509::DEFAULT_CERT_FILE en lugar de la ruta hardcodeada
 /etc/ssl/certs/ca-certificates.crt para soportar múltiples plataformas.

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    arca.rb (1.0.1)
+    arca.rb (1.1.0)
       activesupport
       builder
       httpclient
@@ -39,7 +39,7 @@ GEM
     connection_pool (3.0.2)
     date (3.5.1)
     drb (2.2.3)
-    faraday (2.14.0)
+    faraday (2.14.1)
       faraday-net_http (>= 2.0, < 3.5)
       json
       logger
@@ -58,7 +58,7 @@ GEM
     i18n (1.14.8)
       concurrent-ruby (~> 1.0)
     io-console (0.8.2)
-    json (2.18.0)
+    json (2.18.1)
     language_server-protocol (3.17.0.5)
     lint_roller (1.1.0)
     logger (1.7.0)
@@ -111,7 +111,7 @@ GEM
     reline (0.6.3)
       io-console (~> 0.5)
     rexml (3.4.4)
-    rubocop (1.84.0)
+    rubocop (1.84.1)
       json (~> 2.3)
       language_server-protocol (~> 3.17.0.2)
       lint_roller (~> 1.1.0)

data/LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2025 ARCA Kit
+Copyright (c) 2025 driveton
 
 MIT License
 

data/README.md

@@ -75,7 +75,7 @@ Opción del constructor: `env: :development` o `env: :production` (symbol o stri
 
 ## Uso
 
-Opciones comunes del constructor: `env`, `cuit`, `key`, `cert`. Opcionales: `savon` (hash pasado a Savon), `ta_path` (WSAA), etc.
+Opciones comunes del constructor: `env`, `cuit`, `key`, `cert`. Opcionales: `savon` (hash pasado a Savon), `ta_path` (WSAA, path del archivo TA), `store` (WSAA, objeto con `read`/`write` para guardar TA en backend propio), etc.
 
 ### Cliente de bajo nivel (`Arca::Client`)
 
@@ -112,6 +112,11 @@ wsaa = Arca::WSAA.new(env: :development, cuit: '20123456789', key: key, cert: ce
 auth = wsaa.auth  # => { token: '...', sign: '...' }
 ```
 
+Opcionalmente podés pasar un **store** para guardar el TA en tu propio backend (por ejemplo DB por tenant con cifrado). Sin `store`, se usa un archivo JSON en `tmp/`. El store debe implementar:
+
+- **`read(cuit:, env:, service:)`** — devuelve `nil` o un Hash con claves simbólicas: `:token`, `:sign`, `:generation_time`, `:expiration_time` (Time o string ISO8601).
+- **`write(cuit:, env:, service:, ta:, expires_at:)`** — se invoca después de un `#login` exitoso; `ta` es el hash del TA y `expires_at` el Time de vencimiento.
+
 ### WSFE (Factura Electrónica)
 ```ruby
 ws = Arca::WSFE.new(env: :development, cuit: '20123456789', key: key, cert: cert)

data/arca.gemspec

@@ -12,7 +12,7 @@ Gem::Specification.new do |s|
   s.email       = [ 'hola@arcakit.dev' ]
   s.homepage    = 'https://arcakit.dev'
   s.summary     = 'Cliente Ruby para webservices de AFIP/ARCA: facturación electrónica, comprobantes y servicios tributarios de Argentina'
-  s.description = 'Cliente Ruby para integrar webservices SOAP de ARCA/AFIPen Argentina. Soporta facturación electrónica (WSFE), factura electrónica de crédito MiPyMEs (WSFeCred), autenticación WSAA, consulta de padrón de contribuyentes, constatación de comprobantes fiscales y todos los servicios tributarios de la API de AFIP. Ideal para aplicaciones Rails que necesitan generar facturas electrónicas y comprobantes en Argentina.'
+  s.description = 'Cliente Ruby para integrar webservices SOAP de ARCA/AFIP en Argentina.'
   s.license     = 'MIT'
 
   s.files         = `git ls-files`.split("\n")

data/lib/arca/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Arca
-  VERSION = "1.0.1"
+  VERSION = "1.1.0"
 end

data/lib/arca/wsaa.rb

@@ -4,7 +4,7 @@ require "json"
 
 module Arca
   class WSAA
-    attr_reader :key, :cert, :service, :ta, :cuit, :client, :env
+    attr_reader :key, :cert, :service, :ta, :cuit, :client, :env, :store
 
     WSDL = {
       development: "https://wsaahomo.afip.gov.ar/ws/services/LoginCms?wsdl",
@@ -20,6 +20,7 @@ module Arca
       @ttl = options[:ttl] || 2400
       @cuit = options[:cuit]
       @client = Client.new Hash(options[:savon]).reverse_merge(wsdl: WSDL[@env])
+      @store = options[:store]
       @ta_path = options[:ta_path] || default_ta_path
     end
 
@@ -79,6 +80,10 @@ module Arca
     end
 
     def restore_ta
+      if @store
+        return restore_ta_from_store
+      end
+
       return nil unless File.exist?(@ta_path) && !File.empty?(@ta_path)
 
       data = JSON.parse(File.read(@ta_path))
@@ -92,6 +97,27 @@ module Arca
       nil
     end
 
+    def restore_ta_from_store
+      h = @store.read(cuit: @cuit, env: @env, service: @service)
+      return nil if h.nil? || !h.is_a?(Hash)
+
+      {
+        token: h[:token],
+        sign: h[:sign],
+        generation_time: normalize_ta_time(h[:generation_time]),
+        expiration_time: normalize_ta_time(h[:expiration_time])
+      }
+    end
+
+    def normalize_ta_time(value)
+      return nil if value.nil?
+      return value if value.is_a?(Time)
+
+      Time.parse(value.to_s)
+    rescue ArgumentError
+      nil
+    end
+
     def ta_expirado?(ta)
       if ta.nil?
         true
@@ -103,6 +129,25 @@ module Arca
     end
 
     def persist_ta(ta)
+      if @store
+        begin
+          @store.write(
+            cuit: @cuit,
+            env: @env,
+            service: @service,
+            ta: ta,
+            expires_at: ta[:expiration_time]
+          )
+        rescue StandardError => e
+          raise ServerError, e
+        end
+        return
+      end
+
+      persist_ta_to_file(ta)
+    end
+
+    def persist_ta_to_file(ta)
       dir = File.dirname(@ta_path)
       FileUtils.mkdir_p(dir)
 
@@ -116,13 +161,13 @@ module Arca
       temp_path = "#{@ta_path}.#{Process.pid}.tmp"
       File.write(temp_path, JSON.generate(payload))
       File.rename(temp_path, @ta_path)
-    rescue StandardError
+    rescue StandardError => e
       File.delete(temp_path) if defined?(temp_path) && File.exist?(temp_path)
-      raise
+      raise ServerError, e
     end
 
     def xsd_datetime(time)
-      time.strftime("%Y-%m-%dT%H:%M:%S%z").sub /(\d{2})(\d{2})$/, '\1:\2'
+      time.strftime("%Y-%m-%dT%H:%M:%S%z").sub(/(\d{2})(\d{2})$/, '\1:\2')
     end
 
     def from_xsd_datetime(str)

data/test/arca/wsaa_test.rb

@@ -76,5 +76,72 @@ module Arca
       subject.auth
       assert_equal "t2", subject.ta[:token]
     end
+
+    def test_persist_ta_encapsula_errores_en_server_error
+      File.stubs(:write).raises(Errno::EACCES.new("Permission denied"))
+      ta = { token: "t", sign: "s", generation_time: Time.now, expiration_time: Time.now + 60 }
+      ws = WSAA.new(wsdl: Arca::WSFE::WSDL[:test])
+
+      error = assert_raises(ServerError) { ws.send(:persist_ta, ta) }
+      assert_equal Errno::EACCES, error.cause.class
+      assert_match(/Permission denied/, error.cause.message)
+    end
+
+    def test_auth_con_store_usa_read_y_no_llama_login_si_ta_valido
+      Time.stubs(:now).returns(Time.local(2010, 1, 1))
+      ta = { token: "stored-token", sign: "stored-sign", generation_time: Time.now, expiration_time: Time.now + 3600 }
+      store = Object.new
+      store.expects(:read).with(cuit: "20123456789", env: :test, service: "wsfe").returns(ta)
+
+      ws = WSAA.new(cuit: "20123456789", env: :test, service: "wsfe", wsdl: Arca::WSFE::WSDL[:test], store: store)
+      ws.expects(:login).never
+
+      assert_equal({ token: "stored-token", sign: "stored-sign" }, ws.auth)
+    end
+
+    def test_auth_con_store_llama_login_y_write_cuando_read_devuelve_nil
+      Time.stubs(:now).returns(Time.local(2010, 1, 1))
+      ta = { token: "new-token", sign: "new-sign", generation_time: Time.now, expiration_time: Time.now + 60 }
+      store = Object.new
+      store.expects(:read).with(cuit: "20123456789", env: :test, service: "wsfe").returns(nil)
+      store.expects(:write).with(cuit: "20123456789", env: :test, service: "wsfe", ta: ta, expires_at: ta[:expiration_time]).returns(nil)
+
+      ws = WSAA.new(cuit: "20123456789", env: :test, service: "wsfe", wsdl: Arca::WSFE::WSDL[:test], store: store)
+      ws.expects(:login).returns(ta)
+
+      ws.auth
+    end
+
+    def test_auth_con_store_normaliza_tiempos_iso8601_de_read
+      Time.stubs(:now).returns(Time.local(2010, 1, 1))
+      gen_str = "2010-01-01T00:00:00-03:00"
+      exp_str = "2010-01-01T02:00:00-03:00"
+      store = Object.new
+      store.expects(:read).with(cuit: "20123456789", env: :test, service: "wsfe").returns({ token: "t", sign: "s", generation_time: gen_str, expiration_time: exp_str })
+
+      ws = WSAA.new(cuit: "20123456789", env: :test, service: "wsfe", wsdl: Arca::WSFE::WSDL[:test], store: store)
+      ws.expects(:login).never
+
+      ws.auth
+      assert_instance_of(Time, ws.ta[:generation_time])
+      assert_instance_of(Time, ws.ta[:expiration_time])
+      assert_equal(Time.parse(gen_str), ws.ta[:generation_time])
+      assert_equal(Time.parse(exp_str), ws.ta[:expiration_time])
+    end
+
+    def test_persist_ta_con_store_encapsula_errores_en_server_error
+      ta = { token: "t", sign: "s", generation_time: Time.now, expiration_time: Time.now + 60 }
+      store = Class.new do
+        def read(*) nil end
+        def write(*) raise Errno::EACCES, "Permission denied" end
+      end.new
+
+      ws = WSAA.new(cuit: "1", env: :test, service: "wsfe", wsdl: Arca::WSFE::WSDL[:test], store: store)
+      assert_same store, ws.store
+
+      error = assert_raises(ServerError) { ws.send(:persist_ta, ta) }
+      assert_equal Errno::EACCES, error.cause.class
+      assert_match(/Permission denied/, error.cause.message)
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: arca.rb
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.1.0
 platform: ruby
 authors:
 - Arca Kit
@@ -149,11 +149,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 2.15.0
-description: Cliente Ruby para integrar webservices SOAP de ARCA/AFIPen Argentina.
-  Soporta facturación electrónica (WSFE), factura electrónica de crédito MiPyMEs (WSFeCred),
-  autenticación WSAA, consulta de padrón de contribuyentes, constatación de comprobantes
-  fiscales y todos los servicios tributarios de la API de AFIP. Ideal para aplicaciones
-  Rails que necesitan generar facturas electrónicas y comprobantes en Argentina.
+description: Cliente Ruby para integrar webservices SOAP de ARCA/AFIP en Argentina.
 email:
 - hola@arcakit.dev
 executables: