Erreur de base de données Discourse lors de la création de nouveaux utilisateurs via la connexion SSO

Eingang · Septembre 2, 2019, 10:14

Résumé

Erreur de contrainte de base de données Discourse due à des valeurs de clé dupliquées générées lorsque des utilisateurs sans compte tentent de se connecter via SSO. Cela provoque l’apparition d’une erreur de serveur interne (500) via un proxy externe.

Discussion

Le 28 ou 29 août, j’ai mis à jour Discourse vers la dernière version bêta (v2.4.0.beta2) ainsi que tous les plug-ins Discourse installés. À peu près au même moment, j’ai également mis à jour SeAT et reconfiguré Discourse pour utiliser un socket UNIX, auquel Apache externe peut faire proxy ou reverse proxy. Auparavant, les personnes pouvaient se connecter via le SSO EvE Online de SeAT et de nouveaux comptes utilisateurs Discourse étaient créés pour elles si elles n’en avaient pas déjà un.

Depuis lors, aucun nouveau compte utilisateur n’a été créé avec succès. À la place, les trois personnes qui ont essayé ont généré des erreurs du type : Échec de la création ou de la recherche de l'utilisateur : ERREUR : la valeur de clé dupliquée viole la contrainte "idx_category_users_user_id_category_id" DÉTAIL : La clé (user_id, category_id)=(36, 6) existe déjà. L’category_id est toujours 6, mais le user_id varie entre 33 et 44, avec même des répétitions de nombres.

En supposant que category_id fait référence aux entrées dans categories, alors 6 semble correspondre à Grass Clippings. En supposant que user_id fait référence aux entrées dans users, nous n’avons aucune entrée supérieure à 32.

La catégorie Grass Clippings est restreinte, au moins en lecture, aux personnes appartenant aux groupes 2GTHRDiscourse, 2GTHRHighCouncil et GRoLLake. Au moins l’une des personnes tentant de se connecter avait une commande add_groups incluant ces trois groupes. La première personne n’avait pas encore l’un des trois groupes, tandis que la personne du milieu en avait au moins un. Elles génèrent toutes la même erreur de base de données avec des détails identiques, sauf pour le user_id.

Les personnes disposant de comptes Discourse créés en se connectant via SSO avant les mises à jour ne semblent pas rencontrer de problèmes de connexion.

Actions entreprises ou examens effectués

Désactivation de force https et retour à un proxy Apache externe vers les ports nginx.
Sans surprise, cela n’a eu aucun effet.
Débogage et correction des problèmes SSO courants
Attention au double encodage URL de la charge utile SSO
Suivi de la trace d’appel et constatation que Discourse atteignait models/discourse_single_sign_on.rb à la ligne 203 de match_email_or_create_user(), où il appelle User.create!(). Malheureusement, la journalisation des paramètres, etc., se fait après ce point, ce qu’il n’atteint jamais car le traitement s’arrête et renvoie une erreur HTTP 500. J’ai vérifié shared/log/rails/production.log.* au cas où.

Autres commentaires

SeAT : Bien que le plug-in SeAT pour Discourse n’ait pas été mis à jour, de nombreux composants de SeAT l’ont été. Il est possible que quelque chose ait changé là-dedans. Cependant, l’erreur se situe clairement du côté de Discourse lors de la tentative de création ou de recherche d’un utilisateur inexistant, donc je suis quelque peu sceptique quant au fait que le problème vienne de SeAT.
Discourse : Je ne me souviens pas de quelle version bêta de Discourse j’exécutais avant la mise à jour. Je suppose qu’il s’était écoulé au moins un ou deux mois depuis ma dernière mise à jour.

Détails supplémentaires

Erreur

Échec de la création ou de la recherche de l'utilisateur : ERREUR : la valeur de clé dupliquée viole la contrainte d'unicité "idx_category_users_user_id_category_id"
DÉTAIL : La clé (user_id, category_id)=(36, 6) existe déjà.
.    add_groups : CORPBLUEP,2GTHRAllianceMember,2GTHRALL,2GTHRHighCouncil,2GTHRCorpDirector,2GTHROfficer,DoWCitadelGunner,HCDIM,CORPDIRBLUEP,2GTHRCorpRecruiter,CORPDIRLAWNS,CORPDIR123HC,GRoLLake,DoWCapital,DoWFax,User,CORPHRBLUEP,2GTHRDiscourse
admin : 
modérateur : 
avatar_force_update : 
avatar_url : http://image.eveonline.com/Character/740426190_128.jpg
bio : 
card_background_url : 
email : something@somewhere.com
external_id : 740426190
groups : 
locale : 
locale_force_update : 
name : Mindframe
nonce : e6f9da9e62b720a1efd97348ca20f101
profile_background_url : 
remove_groups : Superuser,SPRKYAllianceMember,DoWFieldMarshall,DoWScoutCoordinator,DoWFCGang,DoWFCAlliance,DoWFCSenior,DoWScout,DoWSuper,DoWTitan,DoWFC,2GTHRCorpCEO,DoWTitanBridging,DoLLogisticsSov,DoLLogisticsShipping,HCDoW,HCDoL,HCDoC,HCDoF,DoCDoIT,DoCTechnoMage,DoCDiplomat,DoCDepartmentLead,DoCRecruiterPilots,DoCRecruiterCorporations,DoCDiplomatLead,DoWBLOP,ViewGroups,CORPAIEU,CORPDIRAIEU,CORPZBMC,CORPC.R.Y,CORP123HC,CORPALTBH,CORPLAWNS,CORPDIRZBMC,CORPDIRC.R.Y,CORPDIRALTBH,CORPF3R4L,CORPDIRF3R4L,CORPSPRKY,CORPDIRSPRKY,CORPRMKR,CORPDIRRMKR,CORPHR123HC,CORPHRAIEU,CORPHRZBMC,CORPHRC.R.Y,CORPHRF3R4L,SPRKYDiscourse,CORPHRLAWNS,CORPDIROUSYN,CORPHROUSYN,CORPOUSYN,DoWReimbursementTeam,DoWCarrier,DoWDread,WCMember,WCALL,WCMindframesMarketVisitors
require_activation : 
return_sso_url : 
suppress_welcome_message : 
title : 
username : Mindframe
website :     /var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-mini-profiler-1.0.2/lib/patches/db/pg.rb:110:in `async_exec'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-mini-profiler-1.0.2/lib/patches/db/pg.rb:110:in `async_exec'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/mini_sql-0.2.2/lib/mini_sql/postgres/connection.rb:118:in `run'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/mini_sql-0.2.2/lib/mini_sql/postgres/connection.rb:90:in `exec'
/var/www/discourse/app/models/user.rb:1403:in `set_default_categories_preferences'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/callbacks.rb:426:in `block in make_lambda'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/callbacks.rb:236:in `block in halting_and_conditional'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/callbacks.rb:517:in `block in invoke_after'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/callbacks.rb:517:in `each'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/callbacks.rb:517:in `invoke_after'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/callbacks.rb:133:in `run_callbacks'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/callbacks.rb:816:in `_run_create_callbacks'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/callbacks.rb:346:in `_create_record'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/timestamp.rb:102:in `_create_record'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/persistence.rb:705:in `create_or_update'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/callbacks.rb:342:in `block in create_or_update'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/callbacks.rb:132:in `run_callbacks'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/callbacks.rb:816:in `_run_save_callbacks'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/callbacks.rb:342:in `create_or_update'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/persistence.rb:308:in `save!'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/validations.rb:52:in `save!'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/transactions.rb:315:in `block in save!'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/transactions.rb:387:in `block in with_transaction_returning_status'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/connection_adapters/abstract/database_statements.rb:267:in `block in transaction'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/connection_adapters/abstract/transaction.rb:239:in `block in within_new_transaction'
/usr/local/lib/ruby/2.6.0/monitor.rb:230:in `mon_synchronize'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/connection_adapters/abstract/transaction.rb:236:in `within_new_transaction'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/connection_adapters/abstract/database_statements.rb:267:in `transaction'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/transactions.rb:212:in `transaction'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/transactions.rb:385:in `with_transaction_returning_status'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/transactions.rb:315:in `save!'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/suppressor.rb:48:in `save!'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/persistence.rb:53:in `create!'
/var/www/discourse/app/models/discourse_single_sign_on.rb:203:in `block in match_email_or_create_user'
/var/www/discourse/lib/distributed_mutex.rb:31:in `block in synchronize'
/var/www/discourse/lib/distributed_mutex.rb:27:in `synchronize'
/var/www/discourse/lib/distributed_mutex.rb:27:in `synchronize'
/var/www/discourse/lib/distributed_mutex.rb:12:in `synchronize'
/var/www/discourse/app/models/discourse_single_sign_on.rb:186:in `match_email_or_create_user'
/var/www/discourse/app/models/discourse_single_sign_on.rb:74:in `lookup_or_create_user'
/var/www/discourse/app/controllers/session_controller.rb:169:in `sso_login'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal/basic_implicit_render.rb:6:in `send_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/abstract_controller/base.rb:194:in `process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal/rendering.rb:30:in `process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/abstract_controller/callbacks.rb:42:in `block in process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/callbacks.rb:132:in `run_callbacks'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/abstract_controller/callbacks.rb:41:in `process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal/rescue.rb:22:in `process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal/instrumentation.rb:34:in `block in process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/notifications.rb:168:in `block in instrument'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/notifications/instrumenter.rb:23:in `instrument'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/notifications.rb:168:in `instrument'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal/instrumentation.rb:32:in `process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal/params_wrapper.rb:256:in `process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/railties/controller_runtime.rb:24:in `process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/abstract_controller/base.rb:134:in `process'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionview-5.2.3/lib/action_view/rendering.rb:32:in `process'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-mini-profiler-1.0.2/lib/mini_profiler/profiling_methods.rb:78:in `block in profile_method'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal.rb:191:in `dispatch'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal.rb:252:in `dispatch'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/routing/route_set.rb:52:in `dispatch'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/routing/route_set.rb:34:in `serve'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/journey/router.rb:52:in `block in serve'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/journey/router.rb:35:in `each'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/journey/router.rb:35:in `serve'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/routing/route_set.rb:840:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-protection-2.0.7/lib/rack/protection/frame_options.rb:31:in `call'
/var/www/discourse/lib/middleware/omniauth_bypass_middleware.rb:68:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/tempfile_reaper.rb:15:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/conditional_get.rb:25:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/head.rb:12:in `call'
/var/www/discourse/lib/content_security_policy/middleware.rb:12:in `call'
/var/www/discourse/lib/middleware/anonymous_cache.rb:219:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/session/abstract/id.rb:232:in `context'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/session/abstract/id.rb:226:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/cookies.rb:670:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/callbacks.rb:28:in `block in call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/callbacks.rb:98:in `run_callbacks'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/callbacks.rb:26:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/debug_exceptions.rb:61:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/show_exceptions.rb:33:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/logster-2.3.2/lib/logster/middleware/reporter.rb:43:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/railties-5.2.3/lib/rails/rack/logger.rb:38:in `call_app'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/railties-5.2.3/lib/rails/rack/logger.rb:28:in `call'
/var/www/discourse/config/initializers/100-quiet_logger.rb:18:in `call'
/var/www/discourse/config/initializers/100-silence_logger.rb:31:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/remote_ip.rb:81:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/request_id.rb:27:in `call'
/var/www/discourse/lib/middleware/enforce_hostname.rb:17:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/method_override.rb:22:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/executor.rb:14:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/sendfile.rb:111:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-mini-profiler-1.0.2/lib/mini_profiler/profiler.rb:171:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/message_bus-2.2.2/lib/message_bus/rack/middleware.rb:57:in `call'
/var/www/discourse/lib/middleware/request_tracker.rb:169:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/railties-5.2.3/lib/rails/engine.rb:524:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/railties-5.2.3/lib/rails/railtie.rb:190:in `public_send'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/railties-5.2.3/lib/rails/railtie.rb:190:in `method_missing'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/urlmap.rb:68:in `block in call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/urlmap.rb:53:in `each'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/urlmap.rb:53:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/unicorn-5.5.1/lib/unicorn/http_server.rb:605:in `process_client'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/unicorn-5.5.1/lib/unicorn/http_server.rb:700:in `worker_loop'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/unicorn-5.5.1/lib/unicorn/http_server.rb:548:in `spawn_missing_workers'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/unicorn-5.5.1/lib/unicorn/http_server.rb:144:in `start'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/unicorn-5.5.1/bin/unicorn:128:in `<top (required)>'
/var/www/discourse/vendor/bundle/ruby/2.6.0/bin/unicorn:23:in `load'
/var/www/discourse/vendor/bundle/ruby/2.6.0/bin/unicorn:23:in `<main>'

sam · Septembre 3, 2019, 4:35

Cela concerne ce code :

github.com/discourse/discourse

app/models/user.rb

b9954b53b


      
          def set_default_categories_preferences
            return if self.staged?
          
            values = []
          
            %w{watching watching_first_post tracking muted}.each do |s|
              category_ids = SiteSetting.get("default_categories_#{s}").split("|").map(&:to_i)
              category_ids.each do |category_id|
                next if category_id == 0
                values << "(#{self.id}, #{category_id}, #{CategoryUser.notification_levels[s.to_sym]})"
              end
            end
          
            if values.present?
              DB.exec("INSERT INTO category_users (user_id, category_id, notification_level) VALUES #{values.join(",")}")
            end
          end

Ce qui s’est produit ici, c’est que nous avons modifié l’index sur category_user comme suit :

github.com/discourse/discourse

app/models/category_user.rb

b9954b53b


      
          #  idx_category_users_category_id_user_id  (category_id,user_id) UNIQUE
          #  idx_category_users_user_id_category_id  (user_id,category_id) UNIQUE

Avez-vous un groupe qui est à la fois par défaut suivi et par défaut tracé dans les paramètres du site ?

Regardez : default categories watching, default categories tracking, default categories muted et default categories watching first post.

Voici ce qui corrige le problème :

diff --git a/app/models/user.rb b/app/models/user.rb
index c1a94949a6..85b2ca9244 100644
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -1390,10 +1390,15 @@ class User < ActiveRecord::Base
     return if self.staged?
 
     values = []
+    # allouer l'ensemble plus tard
+    seen = nil
 
     %w{watching watching_first_post tracking muted}.each do |s|
       category_ids = SiteSetting.get("default_categories_#{s}").split("|").map(&:to_i)
       category_ids.each do |category_id|
+        seen ||= Set.new
+        next if seen.include?(category_id)
+        seen << category_id
         next if category_id == 0
         values << "(#{self.id}, #{category_id}, #{CategoryUser.notification_levels[s.to_sym]})"
       end
diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb
index cc50d88b2e..4075ee6194 100644
--- a/spec/models/user_spec.rb
+++ b/spec/models/user_spec.rb
@@ -1603,8 +1603,12 @@ describe User do
 
       SiteSetting.default_categories_watching = category0.id.to_s
       SiteSetting.default_categories_tracking = category1.id.to_s
-      SiteSetting.default_categories_muted = category2.id.to_s
+
+      # ceci est invalide, mais nous ne validons pas, alors assurons-nous que rien ne casse
+      SiteSetting.default_categories_muted = "#{category2.id}|#{category0.id}"
+
       SiteSetting.default_categories_watching_first_post = category3.id.to_s
+
     end
 
     it "has overriden preferences" do

Mais je ne suis pas fan de cette correction ; les paramètres du site devraient valider qu’il n’y a pas de chevauchement lors de la sauvegarde, et nous devrions migrer les mauvaises données.

@daniel Je pense que c’est toi qui as introduit cette nouvelle contrainte ici. Peut-être ajouter une validation lorsque les gens définissent le paramètre du site ?

Eingang · Septembre 3, 2019, 5:58

J’ai des catégories suivies et des catégories surveillées, mais il n’y a pas de chevauchement entre elles. Ce sont toutes des catégories de premier niveau. Cependant, j’ai un chevauchement entre default categories watching et default categories watching first post. Une catégorie est présente dans les deux, et cette catégorie correspond à l’ID de la catégorie mentionnée dans mon erreur de contrainte de base de données. Cela pourrait-il faire partie du problème ?

Eingang · Septembre 3, 2019, 6:11

Je me réponds : En retirant la catégorie de default categories watching first post afin qu’elle ne soit présente que dans default categories watching, le problème a été résolu ; une personne sans compte qui ne pouvait pas se connecter auparavant peut désormais le faire.

debryc · Octobre 5, 2021, 11:48

Je viens de rencontrer ce bug.

Les utilisateurs ne pouvaient pas se connecter car les « Notes de recrutement » et les « Notes des partenaires » se répétaient dans default categories regular.

sam · Octobre 6, 2021, 1:58

Un doublon devrait être très facile à prévenir lors de l’enregistrement du paramètre. Je ne suis pas sûr en revanche pour la correction de l’historique.

@vinothkannans, peux-tu t’assurer que, lors de l’enregistrement, ces paramètres par défaut nettoient correctement les données ?

Supprimer les doublons
Si vous enregistrez une « surveillance par défaut » pour la catégorie A… et qu’un « suivi par défaut » est déjà défini pour A, renvoyer une erreur en expliquant le problème

debryc · Octobre 6, 2021, 3:31

Ma meilleure hypothèse sur la façon dont cela s’est produit est la suivante :

Création de « Partner Notes » en tant que catégorie distincte et définition de celle-ci comme catégorie par défaut (Regular)
Création de « Recruitment Notes » en tant que catégorie distincte et définition de celle-ci comme catégorie par défaut (Regular)
Création de « Organizers » en tant que catégorie distincte et définition de celle-ci comme catégorie par défaut (Regular)
Modification des paramètres de « Partner Notes » et de « Recruitment Notes » pour définir leur catégorie parente sur « Organizers » et (je pense) aucune modification apportée au paramètre « Default Categories Regular »

Je soutiens fermement la correction de ce bug, car l’impossibilité pour les nouveaux utilisateurs de s’inscrire équivaut à une catastrophe et est difficile à détecter, car les nouveaux utilisateurs abandonneraient probablement simplement sans contacter qui que ce soit pour obtenir de l’aide.

En attendant, peut-être que le message d’erreur pourrait inclure : « Si vous continuez à rencontrer des problèmes lors de l’inscription, veuillez contacter site email pour obtenir de l’aide ? »

Sujet		Réponses	Vues
Error: Users can't make new accounts Bug	12	2004	Juillet 6, 2023
User cannot signup due to duplicate values in category notification states Self-hosting	2	734	Septembre 4, 2019
Signup error: PG::UniqueViolation (ERROR: duplicate key value violates unique constraint Support	5	2188	Mars 30, 2023
SSO Broken - The requested URL or resource could not be found SSO	8	1385	Juillet 22, 2022
Unable to create categories Bug docker	2	1216	Avril 16, 2015