about summary refs log tree commit diff
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2021-03-19 17:15:36 +0100
committerGitHub <noreply@github.com>2021-03-19 17:15:36 +0100
commitaf8fe6e1e9d728a4af39941e1bde918b0ca7d204 (patch)
treebb33bb3aa56cb0b7b54caf14151c5b1ce1666c82
parent741d0952b174740e70a09fe6db6862624dfe1e44 (diff)
WIP (#15222)
-rw-r--r--app/models/concerns/omniauthable.rb1
-rw-r--r--app/models/user.rb2
2 files changed, 1 insertions, 2 deletions
diff --git a/app/models/concerns/omniauthable.rb b/app/models/concerns/omniauthable.rb
index 79d671d10..791a94911 100644
--- a/app/models/concerns/omniauthable.rb
+++ b/app/models/concerns/omniauthable.rb
@@ -68,7 +68,6 @@ module Omniauthable
     def user_params_from_auth(email, auth)
       {
         email: email || "#{TEMP_EMAIL_PREFIX}-#{auth.uid}-#{auth.provider}.com",
-        password: Devise.friendly_token[0, 20],
         agreement: true,
         external: true,
         account_attributes: {
diff --git a/app/models/user.rb b/app/models/user.rb
index b4508c2eb..5a149f573 100644
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -468,7 +468,7 @@ class User < ApplicationRecord
   end
 
   def validate_email_dns?
-    email_changed? && !(Rails.env.test? || Rails.env.development?)
+    email_changed? && !external? && !(Rails.env.test? || Rails.env.development?)
   end
 
   def invite_text_required?