katalyst-koi 4.14.2 → 4.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '085801680584e3ab05d9a8ae0562fd5c2bf22fe6d1388b03c3dce053f1a42c66'
-  data.tar.gz: 334214269ecfe218bed7c219923e3b95a94290f532aaa4c8448871ae82ac4862
+  metadata.gz: 727d460171f01afa0e58e2640f5a42b1f23262f9b18d03a8eeeb9fbc2e54c8ea
+  data.tar.gz: db82baad11c408b11cbd1842632acca9d72b476180fef1af70871dadba4f2bfa
 SHA512:
-  metadata.gz: 13190e0428aa5cad654a031f4de6c6089ce25f2d32febbbd1801f826772ec30c75c0e6b30d19d7bfc026ea4f991139b6293f9e50d3126ff46c2742fceefc5623
-  data.tar.gz: 3f1b9282f97ec2c563bf5c9feeef48051b7cc17b617782f6cb35924a1664bf7499b768d81f52134880375f20efd3172dea0d38eab289dd126322393deecec564
+  metadata.gz: cc0df96b3f59832d975518959f57500a68af1535e051365d75f9943e06e13aedc30b7d00c25817ecad213486c560cb71bcc64b6ba6c57f395bb3f81ef46deca5
+  data.tar.gz: 19477da2a8da96c68947a0d6d4903aeece46e91aed0902fa29e3a3abf87c4f4699254a42eb988f623c92e2f3be042995ea1d789c9bff7edb87f7767a8a964c46

data/app/controllers/admin/admin_users_controller.rb

@@ -88,6 +88,8 @@ module Admin
       attribute :email, :string
       attribute :last_sign_in_at, :date
       attribute :sign_in_count, :integer
+      attribute :passkey, :boolean, scope: :has_passkey
+      attribute :mfa, :boolean, scope: :has_otp
     end
   end
 end

data/app/controllers/admin/otps_controller.rb

@@ -0,0 +1,52 @@
+# frozen_string_literal: true
+
+module Admin
+  class OtpsController < ApplicationController
+    before_action :set_admin_user
+
+    def new
+      @admin_user.otp_secret = ROTP::Base32.random
+
+      render :new, locals: { admin: @admin_user }
+    end
+
+    def create
+      @admin_user.otp_secret = otp_params[:otp_secret]
+
+      if @admin_user.otp.verify(otp_params[:token])
+        @admin_user.save
+
+        redirect_to admin_admin_user_path(@admin_user), status: :see_other
+      else
+        @admin_user.errors.add(:token, :invalid)
+
+        respond_to do |format|
+          format.html { redirect_to admin_admin_user_path(@admin_user), status: :see_other }
+          format.turbo_stream { render locals: { admin: @admin_user } }
+        end
+      end
+    end
+
+    def destroy
+      @admin_user.update!(otp_secret: nil)
+
+      redirect_to admin_admin_user_path(@admin_user), status: :see_other
+    end
+
+    private
+
+    def otp_params
+      params.require(:admin).permit(:otp_secret, :token)
+    end
+
+    def set_admin_user
+      @admin_user = Admin::User.find(params[:admin_user_id])
+
+      if current_admin == @admin_user
+        request.variant = :self
+      else
+        head(:forbidden)
+      end
+    end
+  end
+end

data/app/controllers/admin/sessions_controller.rb

@@ -14,15 +14,20 @@ module Admin
     end
 
     def create
-      if (admin_user = webauthn_authenticate! || params_authenticate!)
-        record_sign_in!(admin_user)
-
-        session[:admin_user_id] = admin_user.id
-
-        redirect_to(url_from(params[:redirect].presence) || admin_dashboard_path, status: :see_other)
+      if session_params[:response].present?
+        create_session_with_webauthn
+      elsif session_params[:token].present?
+        create_session_with_token
+      elsif session_params[:password].present?
+        create_session_with_password
+      elsif session_params[:email].present?
+        # conversational flow, ask for password regardless of email
+        admin_user = Admin::User.new(session_params.slice(:email))
+
+        render(:password, status: :unprocessable_content, locals: { admin_user: })
       else
-        admin_user = Admin::User.new(session_params.slice(:email, :password))
-        admin_user.errors.add(:email, "Invalid email or password")
+        # invalid request, re-render new
+        admin_user = Admin::User.new
 
         render(:new, status: :unprocessable_content, locals: { admin_user: })
       end
@@ -38,16 +43,66 @@ module Admin
 
     private
 
+    def create_session_with_password
+      # constant time lookup for user with password verification
+      admin_user = Admin::User.authenticate_by(session_params.slice(:email, :password))
+
+      if admin_user.present? && admin_user.requires_otp?
+        session[:pending_admin_user_id] = admin_user.id
+
+        render(:otp, status: :unprocessable_content, locals: { admin_user: })
+      elsif admin_user.present?
+        admin_sign_in(admin_user)
+      else
+        admin_user = Admin::User.new(session_params.slice(:email, :password))
+        admin_user.errors.add(:email, :invalid)
+
+        render(:new, status: :unprocessable_content, locals: { admin_user: })
+      end
+    end
+
+    def create_session_with_token
+      # assume that the previous step injected the user's ID into the session and remove it regardless of outcome
+      admin_user = Admin::User.find_by(id: session.delete(:pending_admin_user_id))
+
+      if admin_user&.otp&.verify(session_params[:token],
+                                 drift_ahead:  15,
+                                 drift_behind: 15,
+                                 after:        admin_user.current_sign_in_at)
+        admin_sign_in(admin_user)
+      else
+        admin_user = Admin::User.new
+        admin_user.errors.add(:email, :invalid)
+
+        render(:new, status: :unprocessable_content, locals: { admin_user: })
+      end
+    end
+
+    def create_session_with_webauthn
+      if (admin_user = webauthn_authenticate!)
+        admin_sign_in(admin_user)
+      else
+        admin_user = Admin::User.new
+        admin_user.errors.add(:email, :invalid)
+
+        render(:new, status: :unprocessable_content, locals: { admin_user: })
+      end
+    end
+
     def redirect_authenticated
       redirect_to(admin_dashboard_path, status: :see_other)
     end
 
-    def session_params
-      params.require(:admin).permit(:email, :password, :response)
+    def admin_sign_in(admin_user)
+      record_sign_in!(admin_user)
+
+      session[:admin_user_id] = admin_user.id
+
+      redirect_to(url_from(params[:redirect].presence) || admin_dashboard_path, status: :see_other)
     end
 
-    def params_authenticate!
-      Admin::User.authenticate_by(session_params.slice(:email, :password))
+    def session_params
+      params.require(:admin).permit(:email, :password, :token, :response)
     end
 
     def update_last_sign_in(admin_user)

data/app/models/admin/user.rb

@@ -3,6 +3,7 @@
 module Admin
   class User < ApplicationRecord
     include Koi::Model::Archivable
+    include Koi::Model::OTP
 
     def self.model_name
       ActiveModel::Name.new(self, nil, "Admin")
@@ -27,5 +28,26 @@ module Admin
         where("email LIKE :query OR name LIKE :query", query: "%#{query}%")
       end
     end
+
+    scope :has_otp, ->(otp) do
+      if otp
+        where.not(otp_secret: nil)
+      else
+        where(otp_secret: nil)
+      end
+    end
+
+    scope :has_passkey, ->(passkey) do
+      if passkey
+        where(id: Admin::Credential.select(:admin_id))
+      else
+        where.not(id: Admin::Credential.select(:admin_id))
+      end
+    end
+
+    def passkey
+      credentials.any?
+    end
+    alias passkey? passkey
   end
 end

data/app/models/concerns/koi/model/otp.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Koi
+  module Model
+    module OTP
+      extend ActiveSupport::Concern
+
+      included do
+        attribute :token, :string
+      end
+
+      def requires_otp?
+        otp_secret.present?
+      end
+
+      def otp
+        ROTP::TOTP.new(otp_secret) if otp_secret.present?
+      end
+    end
+  end
+end

data/app/views/admin/admin_users/index.html.erb

@@ -18,6 +18,9 @@
   <% row.boolean :credentials, label: "Passkey" do |cell| %>
     <%= cell.value.any? ? "Yes" : "No" %>
   <% end %>
+  <% row.boolean :otp, label: "MFA" do |cell| %>
+    <%= cell.value.present? ? "Yes" : "No" %>
+  <% end %>
 <% end %>
 
 <%= table_pagination_with(collection:) %>

data/app/views/admin/admin_users/show.html+self.erb

@@ -4,11 +4,25 @@
   <%= render Koi::Header::ShowComponent.new(resource: admin) %>
 <% end %>
 
-<%= render Koi::SummaryListComponent.new(model: admin, class: "item-table") do |builder| %>
+<%= render Koi::SummaryTableComponent.new(model: admin) do |builder| %>
   <%= builder.text :name %>
   <%= builder.text :email %>
   <%= builder.date :created_at %>
-  <%= builder.date :last_sign_in_at, label: { text: "Last sign in" } %>
+  <%= builder.date :last_sign_in_at, label: "Last sign in" %>
+  <%= builder.boolean :passkey %>
+  <%= builder.boolean :otp, label: "MFA" do |otp| %>
+    <span class="repel">
+      <%= otp %>
+      <% if otp.value %>
+        <%= button_to("Remove", admin_admin_user_otp_path(admin),
+                      class:  "button button--text",
+                      method: :delete,
+                      form:   { data: { turbo_confirm: "Are you sure?" } }) %>
+      <% else %>
+        <%= kpop_link_to "Add", new_admin_admin_user_otp_path(admin) %>
+      <% end %>
+    </span>
+  <% end %>
 <% end %>
 
 <div class="repel">

data/app/views/admin/admin_users/show.html.erb

@@ -4,18 +4,16 @@
   <%= render Koi::Header::ShowComponent.new(resource: admin) %>
 <% end %>
 
-<%= render Koi::SummaryListComponent.new(model: admin, class: "item-table") do |builder| %>
+<%= render Koi::SummaryTableComponent.new(model: admin, class: "item-table") do |builder| %>
   <%= builder.text :name %>
   <%= builder.text :email %>
   <%= builder.date :created_at %>
-  <%= builder.date :last_sign_in_at, label: { text: "Last sign in" } %>
+  <%= builder.date :last_sign_in_at, label: "Last sign in" %>
+  <%= builder.boolean :passkey %>
+  <%= builder.boolean :otp, label: "MFA" %>
   <%= builder.boolean :archived? %>
 <% end %>
 
-<h3>Passkeys</h3>
-
-<%= render "admin/credentials/credentials", admin: %>
-
 <div class="actions">
   <% if admin.archived? %>
     <%= button_to "Delete", admin_admin_user_path(admin),

data/app/views/admin/otps/_form.html.erb

@@ -0,0 +1,31 @@
+<%# locals: (admin:) %>
+
+<%= form_with(id:     dom_id(admin, :otp),
+              model:  admin,
+              url:    admin_admin_user_otp_path(admin),
+              method: :post,
+              class:  "flow") do |form| %>
+  <section class="flow prose">
+    <p>MFA protects your account by requiring you to enter a six-digit
+      token that changes every 30 seconds. If someone knows or guesses your
+      password they also need to know the current token to log in.</p>
+    <p>In general, we recommend using Passkeys over MFA. Passkeys offer better
+      security than a password + MFA, and they are easier to use.</p>
+    <p><strong>Add an MFA authenticator to your account</strong></p>
+    <ol class="flow">
+      <li>Install an MFA app. Most password managers support MFA.</li>
+      <li>Scan this code using your mobile device or password manager:<br>
+        <%== RQRCode::QRCode.new(admin.otp.provisioning_uri(admin.email)).as_svg(
+               color:           "000",
+               shape_rendering: "crispEdges",
+               module_size:     3,
+               use_path:        true,
+             ) %>
+      </li>
+      <li>Enter the token shown in your app into the field below:</li>
+    </ol>
+  </section>
+  <%= form.hidden_field :otp_secret %>
+  <%= form.govuk_text_field :token %>
+  <%= form.admin_save %>
+<% end %>

data/app/views/admin/otps/create.turbo_stream.erb

@@ -0,0 +1,5 @@
+<%# locals: (admin:) %>
+
+<%= turbo_stream.replace(dom_id(admin, :otp)) do %>
+  <%= render("form", admin:) %>
+<% end %>

data/app/views/admin/otps/new.html.erb

@@ -0,0 +1,5 @@
+<%# locals: (admin:) %>
+
+<%= render Kpop::ModalComponent.new(title: "Configure MFA") do %>
+  <%= render("form", admin:) %>
+<% end %>

data/app/views/admin/sessions/new.html.erb

@@ -1,4 +1,7 @@
+<%# locals: (admin_user:) %>
+
 <%= render "layouts/koi/navigation_header" %>
+
 <%= form_with(
       model: admin_user,
       url:   admin_session_path,
@@ -6,7 +9,7 @@
         controller:                            "webauthn-authentication",
         webauthn_authentication_options_value: { publicKey: webauthn_auth_options },
       },
-    ) do |f| %>
+    ) do |form| %>
   <% (redirect = flash[:redirect] || params[:redirect]) && flash.delete(:redirect) %>
   <% unless flash.empty? %>
     <div class="govuk-error-summary">
@@ -17,15 +20,12 @@
       </ul>
     </div>
   <% end %>
-  <%= f.govuk_fieldset legend: nil do %>
-    <%# note, autocomplete off is ignored by browsers but required by PCI-DSS %>
-    <%= f.govuk_email_field :email, autofocus: true, autocomplete: "off" %>
-    <%= f.govuk_password_field :password, autocomplete: "off" %>
-    <%= f.hidden_field :response, data: { webauthn_authentication_target: "response" } %>
-    <%= hidden_field_tag(:redirect, redirect) %>
-  <% end %>
+  <%# note, autocomplete off is ignored by browsers but required by PCI-DSS %>
+  <%= form.govuk_email_field :email, autofocus: true, autocomplete: "off" %>
+  <%= form.hidden_field :response, data: { webauthn_authentication_target: "response" } %>
+  <%= hidden_field_tag(:redirect, redirect) %>
   <div class="actions-group">
-    <%= f.admin_save "Log in" %>
-    <%= f.button "🔑", type: :button, class: "button button--secondary", data: { action: "webauthn-authentication#authenticate" } %>
+    <%= form.admin_save "Next" %>
+    <%= form.button "🔑", type: :button, class: "button button--secondary", data: { action: "webauthn-authentication#authenticate" } %>
   </div>
 <% end %>

data/app/views/admin/sessions/otp.html.erb

@@ -0,0 +1,10 @@
+<%# locals: (admin_user:) %>
+
+<%= render "layouts/koi/navigation_header" %>
+
+<%= form_with(model: admin_user, url: admin_session_path, method: :post) do |form| %>
+  <%# note, autocomplete off is ignored by browsers but required by PCI-DSS %>
+  <%= form.govuk_text_field :token, autofocus: true, autocomplete: "off" %>
+  <%= hidden_field_tag(:redirect, params[:redirect]) %>
+  <%= form.admin_save "Next" %>
+<% end %>

data/app/views/admin/sessions/password.html.erb

@@ -0,0 +1,11 @@
+<%# locals: (admin_user:) %>
+
+<%= render "layouts/koi/navigation_header" %>
+
+<%= form_with(model: admin_user, url: admin_session_path) do |form| %>
+  <%= form.hidden_field(:email) %>
+  <%# note, autocomplete off is ignored by browsers but required by PCI-DSS %>
+  <%= form.govuk_password_field :password, autofocus: true, autocomplete: "off" %>
+  <%= hidden_field_tag(:redirect, params[:redirect]) %>
+  <%= form.admin_save "Next" %>
+<% end %>

data/config/initializers/inflections.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+ActiveSupport::Inflector.inflections do |inflect|
+  inflect.acronym "MFA"
+  inflect.acronym "OTP"
+end

data/config/locales/koi.en.yml

@@ -14,6 +14,11 @@ en:
     labels:
       new: New
       search: Search
+  activerecord:
+    errors:
+      models:
+        admin:
+          invalid: "Invalid login credentials"
   helpers:
     hint:
       default:

data/config/routes.rb

@@ -10,6 +10,7 @@ Rails.application.routes.draw do
     resources :url_rewrites
     resources :admin_users do
       resources :credentials, only: %i[new create destroy]
+      resource :otp, only: %i[new create destroy]
       resources :tokens, only: %i[create]
       get :archived, on: :collection
       put :archive, on: :collection

data/db/migrate/20241214060913_add_otp_secret_to_admin_users.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class AddOTPSecretToAdminUsers < ActiveRecord::Migration[8.0]
+  def change
+    add_column :admins, :otp_secret, :string
+  end
+end

data/lib/koi/engine.rb

@@ -7,6 +7,8 @@ require "katalyst/kpop"
 require "katalyst/navigation"
 require "katalyst/tables"
 require "pagy"
+require "rotp"
+require "rqrcode"
 require "stimulus-rails"
 require "turbo-rails"
 require "webauthn"

data/lib/koi/form/elements/file_element.rb

@@ -69,7 +69,7 @@ module Koi
         end
 
         def preview_url
-          preview? ? main_app.url_for(value) : ""
+          preview? ? main_app.polymorphic_path(value, only_path: true) : ""
         end
 
         def preview?

data/spec/factories/admins.rb

@@ -5,5 +5,6 @@ FactoryBot.define do
     email { Faker::Internet.email }
     name { Faker::Name.name }
     password { Faker::Internet.password }
+    otp_secret { ROTP::Base32.random }
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: katalyst-koi
 version: !ruby/object:Gem::Version
-  version: 4.14.2
+  version: 4.15.0
 platform: ruby
 authors:
 - Katalyst Interactive
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-11-26 00:00:00.000000000 Z
+date: 2024-12-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -80,6 +80,34 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rotp
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rqrcode
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: webauthn
   requirement: !ruby/object:Gem::Requirement
@@ -343,6 +371,7 @@ files:
 - app/controllers/admin/caches_controller.rb
 - app/controllers/admin/credentials_controller.rb
 - app/controllers/admin/dashboards_controller.rb
+- app/controllers/admin/otps_controller.rb
 - app/controllers/admin/sessions_controller.rb
 - app/controllers/admin/tokens_controller.rb
 - app/controllers/admin/url_rewrites_controller.rb
@@ -363,6 +392,7 @@ files:
 - app/models/admin/user.rb
 - app/models/application_record.rb
 - app/models/concerns/koi/model/archivable.rb
+- app/models/concerns/koi/model/otp.rb
 - app/models/url_rewrite.rb
 - app/views/admin/admin_users/_fields.html+self.erb
 - app/views/admin/admin_users/_fields.html.erb
@@ -378,7 +408,12 @@ files:
 - app/views/admin/credentials/destroy.turbo_stream.erb
 - app/views/admin/credentials/new.html.erb
 - app/views/admin/dashboards/show.html.erb
+- app/views/admin/otps/_form.html.erb
+- app/views/admin/otps/create.turbo_stream.erb
+- app/views/admin/otps/new.html.erb
 - app/views/admin/sessions/new.html.erb
+- app/views/admin/sessions/otp.html.erb
+- app/views/admin/sessions/password.html.erb
 - app/views/admin/shared/icons/_close.html.erb
 - app/views/admin/shared/icons/_cross.html.erb
 - app/views/admin/shared/icons/_menu.html.erb
@@ -418,6 +453,7 @@ files:
 - config/importmap.rb
 - config/initializers/extensions.rb
 - config/initializers/flipper.rb
+- config/initializers/inflections.rb
 - config/locales/koi.en.yml
 - config/locales/pagy.en.yml
 - config/routes.rb
@@ -428,6 +464,7 @@ files:
 - db/migrate/20230531063707_update_admin_users.rb
 - db/migrate/20230602033610_add_archived_to_admin_users.rb
 - db/migrate/20231211005214_add_status_code_to_url_rewrites.rb
+- db/migrate/20241214060913_add_otp_secret_to_admin_users.rb
 - db/seeds.rb
 - lib/generators/koi/active_record/active_record_generator.rb
 - lib/generators/koi/admin/USAGE