collavre_slack 0.2.6 → 0.2.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8d470cccf0cf90daff6918a44ff263700803a3a51114003be481d51fc38431f9
-  data.tar.gz: 63c5cbebe3fd89218f494a41530e190d1ba85ca419e1baccc45914a40c6995aa
+  metadata.gz: 2c3b4139e84c2e9f97e305dda87c4c54982b111c1e7eaa72538b586e3d36b57b
+  data.tar.gz: d9fd80c81173f332e5af67a07d0d51dfcff8264bc3a3d202ac4b253fa8cab182
 SHA512:
-  metadata.gz: 982316ca1c8c573d38b8f5c95003da48503153067be902844c9462458df362b46d6c2cc9188134d20cec78426355058ded4a54d155e17e3e966eef16a7646ba5
-  data.tar.gz: 5252ff282030eba366c99c0dbc9c11bb356c824fd65b505d37d0234b51d15fb35be6b68de435713ecd5739573963f0fa5fa689eb09a8cf08113a2132aee476c9
+  metadata.gz: 7762c23a74cb167f65ee346a820bd6b6267f5aad64a7093e853a73d0241b3e78e05721889e1d4ee9e1f00f890f167bf4bde7129e7d9cf563e75a213bd07df9b1
+  data.tar.gz: fa1e173ca870efe960aab55abb6e03c9451a4cc0860345c470dfd41922ad38195f1d2f4abd78bb5f9a5dc04ed61207e62e9675e29a6a1ddb079558095dcf6fe2

data/config/initializers/collavre_slack.rb

@@ -1,7 +1,27 @@
+# Register Slack OAuth keys with the IntegrationSettings registry so the admin
+# UI can surface them and `Collavre::IntegrationSettings::Resolver` can serve
+# values via the DB > ENV > default precedence. Wrapped in `to_prepare` so
+# definitions re-register on code reload in development.
+Rails.application.config.to_prepare do
+  if defined?(Collavre::IntegrationSettings::Registry)
+    Collavre::IntegrationSettings::Registry.instance.register(
+      :slack_client_id,      category: "slack", sensitive: false, requires_restart: true
+    )
+    Collavre::IntegrationSettings::Registry.instance.register(
+      :slack_client_secret,  category: "slack", sensitive: true,  requires_restart: true
+    )
+    Collavre::IntegrationSettings::Registry.instance.register(
+      :slack_signing_secret, category: "slack", sensitive: true,  requires_restart: true
+    )
+    Collavre::IntegrationSettings::Registry.instance.register(
+      :slack_redirect_uri,   category: "slack", sensitive: false, requires_restart: true
+    )
+  end
+end
+
+# Scopes still come from ENV (not registered as a managed integration setting
+# in Phase 1). OAuth credentials are now resolved lazily by Configuration via
+# the Resolver, so they are no longer assigned here.
 CollavreSlack.configure do |config|
-  config.client_id = ENV.fetch("SLACK_CLIENT_ID", config.client_id)
-  config.client_secret = ENV.fetch("SLACK_CLIENT_SECRET", config.client_secret)
-  config.signing_secret = ENV.fetch("SLACK_SIGNING_SECRET", config.signing_secret)
-  config.redirect_uri = ENV.fetch("SLACK_REDIRECT_URI", config.redirect_uri)
   config.scopes = ENV.fetch("SLACK_SCOPES", config.scopes)
 end

data/lib/collavre_slack/configuration.rb

@@ -1,13 +1,51 @@
 module CollavreSlack
+  # Slack engine configuration.
+  #
+  # OAuth credential getters (`client_id`, `client_secret`, `signing_secret`,
+  # `redirect_uri`) resolve through `Collavre::IntegrationSettings::Resolver`,
+  # which honors the DB > ENV > registered default precedence. Writers are
+  # preserved so tests and explicit `CollavreSlack.configure` overrides keep
+  # working: an explicitly-assigned value short-circuits the resolver.
+  #
+  # Fallback to plain `ENV[...]` when the resolver is not usable:
+  #   - `integration_settings` table missing (db:create, db:schema:load,
+  #     assets:precompile, fresh installs)
+  #   - DB connection not established
+  #   - `Collavre::IntegrationSettings` API not present in the installed
+  #     core `collavre` gem (e.g. `USE_COLLAVRE_GEM=true` pinned to a
+  #     version predating this feature)
   class Configuration
-    attr_accessor :client_id, :client_secret, :signing_secret, :redirect_uri, :scopes
+    RESOLVER_KEYS = {
+      client_id:      { registry: :slack_client_id,      env: "SLACK_CLIENT_ID" },
+      client_secret:  { registry: :slack_client_secret,  env: "SLACK_CLIENT_SECRET" },
+      signing_secret: { registry: :slack_signing_secret, env: "SLACK_SIGNING_SECRET" },
+      redirect_uri:   { registry: :slack_redirect_uri,   env: "SLACK_REDIRECT_URI" }
+    }.freeze
+
+    attr_writer :client_id, :client_secret, :signing_secret, :redirect_uri
+    attr_accessor :scopes
 
     def initialize
-      @client_id = ENV.fetch("SLACK_CLIENT_ID", "")
-      @client_secret = ENV.fetch("SLACK_CLIENT_SECRET", "")
-      @signing_secret = ENV.fetch("SLACK_SIGNING_SECRET", "")
-      @redirect_uri = ENV.fetch("SLACK_REDIRECT_URI", "")
       @scopes = ENV.fetch("SLACK_SCOPES", "chat:write,channels:read,channels:history,groups:read,im:read,mpim:read,users:read,users:read.email,reactions:read,reactions:write")
     end
+
+    RESOLVER_KEYS.each do |attr, meta|
+      define_method(attr) do
+        ivar = instance_variable_get(:"@#{attr}")
+        return ivar unless ivar.nil?
+
+        resolve_setting(meta[:registry], meta[:env])
+      end
+    end
+
+    private
+
+    def resolve_setting(registry_key, env_var)
+      return ENV[env_var] unless defined?(Collavre::IntegrationSettings::Resolver)
+
+      Collavre::IntegrationSettings::Resolver.get(registry_key)
+    rescue ActiveRecord::StatementInvalid, ActiveRecord::ConnectionNotEstablished
+      ENV[env_var]
+    end
   end
 end

data/lib/collavre_slack/version.rb

@@ -1,3 +1,3 @@
 module CollavreSlack
-  VERSION = "0.2.6"
+  VERSION = "0.2.7"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: collavre_slack
 version: !ruby/object:Gem::Version
-  version: 0.2.6
+  version: 0.2.7
 platform: ruby
 authors:
 - Collavre