authentication-zero 2.16.7 → 2.16.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2759ec10946a7ca08a682e430a1035c2c6903bcde53cf65b1dd75abeb12e5fd5
-  data.tar.gz: 56097dbc2b99da7c03ffa308a5d69980c63583d0832d7219c77f19a62e6a8671
+  metadata.gz: 22f85e52857c07004eb56d4fdcad96e6b1bd1f709a6038afff84bb4a8378dce2
+  data.tar.gz: 6467a490abffb63134b1f217a054559cf34fbab939a15bdcc10e419acecbc78d
 SHA512:
-  metadata.gz: 3b8a831fbe073cd0770f28c10f22a265fc2af63d520436c11df99da104f20097fbfaf1379025658729295e61485a400f801e01a2b9d596bd242ceac02995e9fb
-  data.tar.gz: 373df629f7a95f12aa0842063ca87b704f68b4dae5edf3a38d37627f5ea50ee5c54d2380479f9959357e34457cfc7efa6c72cd7919757a4d75801b0061ebc623
+  metadata.gz: b53657217a8f8b5376f12d37dc3ef56989a4159f25d1f3fe89e3398288a599ac2f4a1be0975ad98fb1798b7d084b7e3a4f05bf302145559488c22b9df22bc153
+  data.tar.gz: 3622a1349a61da59814b7476d90ef8cf132a50f5624b10b0e3d92da8d1f996785be02b2929728dd79a578692f708af8552b16923202ca72bca81fa6d6875c0a7

data/.github/workflows/CI.yml

@@ -7,9 +7,7 @@
 name: "Generate sample app and run tests"
 on:
   push:
-    branches: ["master"]
-  pull_request:
-    branches: ["master"]
+    branches: ["*"]
 jobs:
   test_html:
     name: 🧪 Run HTML Tests

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## Authentication Zero 2.16.8 ##
+
+* Verify email using identity/email_verification?sid=xxx instead of
+  identity/email_verification/edit?sid=xxx
+
 ## Authentication Zero 2.16.6 ##
 
 * Remove passwordless from api template

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    authentication-zero (2.16.7)
+    authentication-zero (2.16.8)
 
 GEM
   remote: https://rubygems.org/

data/lib/authentication_zero/version.rb

@@ -1,3 +1,3 @@
 module AuthenticationZero
-  VERSION = "2.16.7"
+  VERSION = "2.16.8"
 end

data/lib/generators/authentication/authentication_generator.rb

@@ -135,7 +135,7 @@ class AuthenticationGenerator < Rails::Generators::Base
     end
 
     route "resource :password_reset,     only: [:new, :edit, :create, :update]", namespace: :identity
-    route "resource :email_verification, only: [:edit, :create]", namespace: :identity
+    route "resource :email_verification, only: [:show, :create]", namespace: :identity
     route "resource :email,              only: [:edit, :update]", namespace: :identity
     route "resource  :password, only: [:edit, :update]"
     route "resources :sessions, only: [:index, :show, :destroy]"

data/lib/generators/authentication/templates/controllers/api/identity/email_verifications_controller.rb.tt

@@ -1,9 +1,9 @@
 class Identity::EmailVerificationsController < ApplicationController
-  skip_before_action :authenticate, only: :edit
+  skip_before_action :authenticate, only: :show
 
-  before_action :set_user, only: :edit
+  before_action :set_user, only: :show
 
-  def edit
+  def show
     @user.update!(verified: true); head(:no_content)
   end
 

data/lib/generators/authentication/templates/controllers/html/application_controller.rb.tt

@@ -3,7 +3,7 @@ class ApplicationController < ActionController::Base
   before_action :authenticate
   <%- if options.lockable? %>
   def require_lock(wait: 1.hour, attempts: 10)
-    counter = Kredis.counter("require_lock:#{request.remote_ip}:#{params[:controller]}:#{params[:action]}", expires_in: wait)
+    counter = Kredis.counter("require_lock:#{request.remote_ip}:#{controller_path}:#{action_name}", expires_in: wait)
     counter.increment
 
     if counter.value > attempts

data/lib/generators/authentication/templates/controllers/html/identity/email_verifications_controller.rb.tt

@@ -1,9 +1,9 @@
 class Identity::EmailVerificationsController < ApplicationController
-  skip_before_action :authenticate, only: :edit
+  skip_before_action :authenticate, only: :show
 
-  before_action :set_user, only: :edit
+  before_action :set_user, only: :show
 
-  def edit
+  def show
     @user.update! verified: true
     redirect_to root_path, notice: "Thank you for verifying your email address"
   end

data/lib/generators/authentication/templates/erb/registrations/new.html.erb.tt

@@ -15,7 +15,7 @@
 
   <div>
     <%%= form.label :email, style: "display: block" %>
-    <%%= form.email_field :email, required: true, autofocus: true, autocomplete: "email" %>
+    <%%= form.email_field :email, value: @user.email, required: true, autofocus: true, autocomplete: "email" %>
   </div>
 
   <div>

data/lib/generators/authentication/templates/erb/sessions/passwordlesses/new.html.erb.tt

@@ -5,7 +5,7 @@
 <%%= form_with(url: sessions_passwordless_path) do |form| %>
   <div>
     <%%= form.label :email, style: "display: block" %>
-    <%%= form.email_field :email, required: true, autofocus: true%>
+    <%%= form.email_field :email, required: true, autofocus: true %>
   </div>
 
   <div>

data/lib/generators/authentication/templates/erb/user_mailer/email_verification.html.erb.tt

@@ -7,7 +7,7 @@
 <%- if code_verifiable? -%>
 <strong><%%= @user.verification_code.value %></strong>
 <%- else -%>
-<%%= link_to "Yes, use this email for my account", edit_identity_email_verification_url(sid: @signed_id) %>
+<%%= link_to "Yes, use this email for my account", identity_email_verification_url(sid: @signed_id) %>
 <%- end -%>
 
 <hr>

data/lib/generators/authentication/templates/test_unit/controllers/api/identity/email_verifications_controller_test.rb.tt

@@ -21,14 +21,14 @@ class Identity::EmailVerificationsControllerTest < ActionDispatch::IntegrationTe
   test "should verify email" do
     sid = @user.email_verification_tokens.create.signed_id(expires_in: 2.days)
 
-    get edit_identity_email_verification_url, params: { sid: sid }, headers: default_headers
+    get identity_email_verification_url, params: { sid: sid }, headers: default_headers
     assert_response :no_content
   end
 
   test "should not verify email with expired token" do
     sid_exp = @user.email_verification_tokens.create.signed_id(expires_in: 0.minutes)
 
-    get edit_identity_email_verification_url, params: { sid: sid_exp }, headers: default_headers
+    get identity_email_verification_url, params: { sid: sid_exp }, headers: default_headers
     assert_response :bad_request
     assert_equal "That email verification link is invalid", response.parsed_body["error"]
   end

data/lib/generators/authentication/templates/test_unit/controllers/html/identity/email_verifications_controller_test.rb.tt

@@ -17,14 +17,14 @@ class Identity::EmailVerificationsControllerTest < ActionDispatch::IntegrationTe
   test "should verify email" do
     sid = @user.email_verification_tokens.create.signed_id(expires_in: 2.days)
 
-    get edit_identity_email_verification_url(sid: sid, email: @user.email)
+    get identity_email_verification_url(sid: sid, email: @user.email)
     assert_redirected_to root_url
   end
 
   test "should not verify email with expired token" do
     sid_exp = @user.email_verification_tokens.create.signed_id(expires_in: 0.minutes)
 
-    get edit_identity_email_verification_url(sid: sid_exp, email: @user.email)
+    get identity_email_verification_url(sid: sid_exp, email: @user.email)
 
     assert_redirected_to edit_identity_email_url
     assert_equal "That email verification link is invalid", flash[:alert]

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: authentication-zero
 version: !ruby/object:Gem::Version
-  version: 2.16.7
+  version: 2.16.8
 platform: ruby
 authors:
 - Nixon
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-02-15 00:00:00.000000000 Z
+date: 2023-03-03 00:00:00.000000000 Z
 dependencies: []
 description:
 email: