2.8.0beta2アップデート以降、一部のユーザーが設定（など）にアクセスできない

[elijah (Eli the Bearded)]

今週の初めに 2.7.0.beta2 から 2.8.0.beta2 へアップデートしました。それ以来、管理者アカウントではなく通常のアカウントで /u/ ページにアクセスできなくなっています。代わりに 404 エラーが表示されます。これはデスクトップ（Firefox 84 および 89）でもモバイル（Android でテスト）でも発生し、ログアウトしてキャッシュをクリアしても同じ現象が再現します。

この変更は以下の問題に関連していると考えています：

@jomaxro

ユーザーカードにおける ser-selectable バッジの表示

デフォルトでは、ユーザーカードにはユーザーの信頼レベルバッジが表示され、次に「最高」レベルのバッジが 2 つ表示されます（ゴールドがあればまずゴールド、なければシルバー、なければブロンズ）。ユーザーは、バッジをお気に入りにマークすることで、表示したい 2 つのバッジを選択できるようになりました。

このアップデートのずっと前から discourse-user-card-badges プラグイン（バージョン 0.1 と表示）をインストールしていましたが、この変更と以下のエラーから、プラグインが今回の変更と互換性がないのではないかと思っています。もしそうなら、アップデート時にプラグインを無効化する、または無効化を促すべきではないでしょうか？

activemodel (6.1.3.2) lib/active_model/attribute_methods.rb:469:in `method_missing'
plugins/discourse-user-card-badges/plugin.rb:40:in `block (2 levels) in activate!'
(eval):93:in `_fast_attributes'
active_model_serializers (0.8.4) lib/active_model/serializer.rb:468:in `rescue in attributes'
active_model_serializers (0.8.4) lib/active_model/serializer.rb:455:in `attributes'
active_model_serializers (0.8.4) lib/active_model/serializer.rb:480:in `_serializable_hash'
active_model_serializers (0.8.4) lib/active_model/serializer.rb:359:in `serializable_hash'
active_model_serializers (0.8.4) lib/active_model/serializer.rb:343:in `as_json'
activesupport (6.1.3.2) lib/active_support/json/encoding.rb:22:in `encode'
activesupport (6.1.3.2) lib/active_support/json/encoding.rb:22:in `encode'
activesupport (6.1.3.2) lib/active_support/core_ext/object/json.rb:43:in `to_json'
active_model_serializers (0.8.4) lib/active_model/serializer.rb:331:in `to_json'
multi_json (1.15.0) lib/multi_json/adapters/oj.rb:56:in `dump'
multi_json (1.15.0) lib/multi_json/adapters/oj.rb:56:in `dump'
multi_json (1.15.0) lib/multi_json/adapter.rb:25:in `dump'
multi_json (1.15.0) lib/multi_json.rb:139:in `dump'
app/controllers/application_controller.rb:519:in `render_json_dump'
app/controllers/users_controller.rb:94:in `block (2 levels) in show'
actionpack (6.1.3.2) lib/action_controller/metal/mime_responds.rb:214:in `respond_to'
app/controllers/users_controller.rb:86:in `show'
actionpack (6.1.3.2) lib/action_controller/metal/basic_implicit_render.rb:6:in `send_action'
actionpack (6.1.3.2) lib/abstract_controller/base.rb:228:in `process_action'
actionpack (6.1.3.2) lib/action_controller/metal/rendering.rb:30:in `process_action'
actionpack (6.1.3.2) lib/abstract_controller/callbacks.rb:42:in `block in process_action'
activesupport (6.1.3.2) lib/active_support/callbacks.rb:117:in `block in run_callbacks'
app/controllers/application_controller.rb:395:in `block in with_resolved_locale'
i18n (1.8.10) lib/i18n.rb:314:in `with_locale'
app/controllers/application_controller.rb:395:in `with_resolved_locale'
activesupport (6.1.3.2) lib/active_support/callbacks.rb:126:in `block in run_callbacks'
activesupport (6.1.3.2) lib/active_support/callbacks.rb:137:in `run_callbacks'
actionpack (6.1.3.2) lib/abstract_controller/callbacks.rb:41:in `process_action'
actionpack (6.1.3.2) lib/action_controller/metal/rescue.rb:22:in `process_action'
actionpack (6.1.3.2) lib/action_controller/metal/instrumentation.rb:34:in `block in process_action'
activesupport (6.1.3.2) lib/active_support/notifications.rb:203:in `block in instrument'
activesupport (6.1.3.2) lib/active_support/notifications/instrumenter.rb:24:in `instrument'
activesupport (6.1.3.2) lib/active_support/notifications.rb:203:in `instrument'
actionpack (6.1.3.2) lib/action_controller/metal/instrumentation.rb:33:in `process_action'
actionpack (6.1.3.2) lib/action_controller/metal/params_wrapper.rb:249:in `process_action'
activerecord (6.1.3.2) lib/active_record/railties/controller_runtime.rb:27:in `process_action'
actionpack (6.1.3.2) lib/abstract_controller/base.rb:165:in `process'
actionview (6.1.3.2) lib/action_view/rendering.rb:39:in `process'
rack-mini-profiler (2.3.2) lib/mini_profiler/profiling_methods.rb:85:in `block in profile_method'


actionpack (6.1.3.2) lib/action_controller/metal.rb:190:in `dispatch'
actionpack (6.1.3.2) lib/action_controller/metal.rb:254:in `dispatch'
actionpack (6.1.3.2) lib/action_dispatch/routing/route_set.rb:50:in `dispatch'
actionpack (6.1.3.2) lib/action_dispatch/routing/route_set.rb:33:in `serve'
actionpack (6.1.3.2) lib/action_dispatch/journey/router.rb:50:in `block in serve'
actionpack (6.1.3.2) lib/action_dispatch/journey/router.rb:32:in `each'
actionpack (6.1.3.2) lib/action_dispatch/journey/router.rb:32:in `serve'
actionpack (6.1.3.2) lib/action_dispatch/routing/route_set.rb:842:in `call'
lib/middleware/omniauth_bypass_middleware.rb:71:in `call'
rack (2.2.3) lib/rack/tempfile_reaper.rb:15:in `call'
rack (2.2.3) lib/rack/conditional_get.rb:27:in `call'
rack (2.2.3) lib/rack/head.rb:12:in `call'
actionpack (6.1.3.2) lib/action_dispatch/http/permissions_policy.rb:22:in `call'
lib/content_security_policy/middleware.rb:12:in `call'
lib/middleware/anonymous_cache.rb:355:in `call'
rack (2.2.3) lib/rack/session/abstract/id.rb:266:in `context'
rack (2.2.3) lib/rack/session/abstract/id.rb:260:in `call'
actionpack (6.1.3.2) lib/action_dispatch/middleware/cookies.rb:689:in `call'
actionpack (6.1.3.2) lib/action_dispatch/middleware/callbacks.rb:27:in `block in call'
activesupport (6.1.3.2) lib/active_support/callbacks.rb:98:in `run_callbacks'
actionpack (6.1.3.2) lib/action_dispatch/middleware/callbacks.rb:26:in `call'
actionpack (6.1.3.2) lib/action_dispatch/middleware/actionable_exceptions.rb:18:in `call'
actionpack (6.1.3.2) lib/action_dispatch/middleware/debug_exceptions.rb:29:in `call'
actionpack (6.1.3.2) lib/action_dispatch/middleware/show_exceptions.rb:33:in `call'
logster (2.9.6) lib/logster/middleware/reporter.rb:43:in `call'
railties (6.1.3.2) lib/rails/rack/logger.rb:37:in `call_app'
railties (6.1.3.2) lib/rails/rack/logger.rb:28:in `call'
config/initializers/100-quiet_logger.rb:23:in `call'
config/initializers/100-silence_logger.rb:31:in `call'
actionpack (6.1.3.2) lib/action_dispatch/middleware/remote_ip.rb:81:in `call'
actionpack (6.1.3.2) lib/action_dispatch/middleware/request_id.rb:26:in `call'
lib/middleware/enforce_hostname.rb:23:in `call'
rack (2.2.3) lib/rack/method_override.rb:24:in `call'
actionpack (6.1.3.2) lib/action_dispatch/middleware/executor.rb:14:in `call'
rack (2.2.3) lib/rack/sendfile.rb:110:in `call'
actionpack (6.1.3.2) lib/action_dispatch/middleware/host_authorization.rb:92:in `call'
rack-mini-profiler (2.3.2) lib/mini_profiler/profiler.rb:249:in `call'


message_bus (3.3.6) lib/message_bus/rack/middleware.rb:61:in `call'
lib/middleware/request_tracker.rb:177:in `call'
railties (6.1.3.2) lib/rails/engine.rb:539:in `call'
railties (6.1.3.2) lib/rails/railtie.rb:207:in `public_send'
railties (6.1.3.2) lib/rails/railtie.rb:207:in `method_missing'
rack (2.2.3) lib/rack/urlmap.rb:74:in `block in call'
rack (2.2.3) lib/rack/urlmap.rb:58:in `each'
rack (2.2.3) lib/rack/urlmap.rb:58:in `call'
unicorn (6.0.0) lib/unicorn/http_server.rb:634:in `process_client'
unicorn (6.0.0) lib/unicorn/http_server.rb:732:in `worker_loop'
unicorn (6.0.0) lib/unicorn/http_server.rb:547:in `spawn_missing_workers'
unicorn (6.0.0) lib/unicorn/http_server.rb:143:in `start'
unicorn (6.0.0) bin/unicorn:128:in `<top (required)>'
vendor/bundle/ruby/2.7.0/bin/unicorn:23:in `load'
vendor/bundle/ruby/2.7.0/bin/unicorn:23:in `<main>'

[codinghorror (Jeff Atwood)]

これはサードパーティ製プラグインですか？もしそうなら、プラグインを無効化するための手順はありますか？

[elijah (Eli the Bearded)]

このプラグインは公式のもののようです。クリックすると以下のページに移動します。

GitHub - discourse/discourse-user-card-badges: This plugin allows users to choose one badge with...

そして、プラグインを無効にすることで404エラーが解消されました。私の質問は、これが公式プラグインであり、今後も動作し続けると考えていたため、なぜ手動での対応が必要なのかということです。

[codinghorror (Jeff Atwood)]

この機能は、プラグインからコアへ移行されたものですか？

その場合、プラグインに非推奨の通知を記載すべきですね。どう思いますか、@neil？

[elijah (Eli the Bearded)]

投稿前に確認したところ、プラグインの最終コミットが 2.8.0beta2 リリースよりも新しいことがわかりました。再度確認すると、そのコミットは「bot」ユーザーによるもので、あまり意味のあるものではないかもしれません。

@codinghorror

これは、プラグインからコア機能に移行した機能でしょうか？

詳しく見てみると、完全に同じものではないと思います。バッジはユーザーカードの左下隅に大きな画像として表示されていました。この新しいコア機能は、ユーザーカードの下部に小さなバッジ画像とバッジ名を含む小さなボックスを制御するもののようです。

この投稿は、バッジ機能の初期バージョンを示しています:

Why does eviltrout have a garbage man badge?

OK, I am going to bite. Why does @eviltrout have a garbage man badge? And why does the badge only display on his user card, and not on his user profile page? Or on the badges page? I categorized as a bug since when you click on the badge, it goes to https://meta.discourse.org/badges/120/garbage-man which is an empty page - probably not as intended. I am interested however in custom badges like this - would be interesting to display for example organizational affiliation with details synced fr…

今回の更新前の状態では、選択可能な画像は達成された功績に関連付けられた画像に限られていました。私のフォーラムのすべてのバッジには、ユーザーカードのバッジをより見栄えよくするために、カスタム画像を割り当てています（デフォルトのものは、ええと、「ゴミ」のようです）。

[neil (Neil Lalonde)]

おそらく、UserBadges テーブルに追加された新しい is_favorite カラムが原因でしょう。

SQL もおそらく現在壊れています。

UserBadge.attribute_names.map do |name|
              if name == 'is_favorite'
                "BOOL_OR(user_badges.#{name}) AS is_favorite"
              else
                "MAX(user_badges.#{name}) AS #{name}"
              end
            end

[martin (Martin Brennan)]

プラグインの修正をマージしました：

FIX: image column is now image_upload in core (#19)

Badge テーブルから image カラムが削除されました（https://github.com/discourse/discourse/commit/a23d0f996132746b29c17644a58b3bd00c7144ae）。その後、プラグインの一部を更新しましたが、いくつかの箇所を見落としていました。

@elijah

公式プラグインであり、引き続き動作すると思っていたので、なぜそれが手動ステップである必要があるのか疑問に思っています。

コアの変更がプラグインを壊さないよう努めていますが、時には使用箇所を見落としたり、プラグインのテストカバレッジが不足していたりします（公式プラグインのすべてのテストは、コアに変更を加えるたびに実行されます）。上記のコミットでは、discourse-user-card-badges プラグインにさらにテストを追加し、将来同様の問題が発生しないようにしました。

[elijah (Eli the Bearded)]

@martin

コアの変更がプラグインを壊さないよう努めてはいますが、時には使用箇所を見落としたり、プラグインのテストカバレッジが不足していたりすることもあります（コアに変更を加えるたびに、すべての公式プラグインに対してテストを実行しています）。

ミスは起こり得ることは理解しています。私がそのように書いた当時、このプラグインの機能は新しいコア機能によって重複していると考えていました。その機能をさらに詳しく調べてみて初めて、これらは異なるものであると気づきました。