Dall'aggiornamento 2.8.0beta2 alcuni utenti non riescono ad accedere alle preferenze (ecc.)

[elijah (Eli the Bearded)]

All’inizio di questa settimana ho aggiornato da 2.7.0.beta2 a 2.8.0.beta2. Da allora, il mio account regolare (ma non quello amministratore) non riesce ad accedere a nessuna delle mie pagine /u/. Ricevo invece un errore 404. Questo succede sia su desktop (Firefox 84 e 89) che su mobile (testato su Android), anche dopo essersi disconnessi, cancellare la cache e riprovare.

Sospetto che questo cambiamento sia correlato al problema:

@jomaxro

Visualizzazione del distintivo ser-selectable sulla scheda utente

Di default, la scheda utente mostra il distintivo del livello di fiducia dell’utente, seguito dai due distintivi di livello “più alto”: prima l’oro, se non c’è l’oro allora l’argento, se non c’è l’argento allora il bronzo. Gli utenti possono ora selezionare i 2 distintivi che desiderano visualizzare contrassegnandoli come preferiti.

Ho installato il plugin discourse-user-card-badges (che mostra la versione 0.1) molto prima di questo aggiornamento, e quel cambiamento insieme all’errore qui sotto mi fanno pensare che forse il plugin non sia compatibile con le modifiche apportate? Se è così, non dovrebbe l’aggiornamento disabilitare il plugin o chiedermi di disabilitarlo?

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)]

Si tratta di un plugin di terze parti? Quindi hai una soluzione per disabilitare il plugin?

[elijah (Eli the Bearded)]

Il plugin sembra essere ufficiale. Cliccandoci sopra, vengo reindirizzato a:

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

E sì, disabilitando il plugin il problema 404 viene risolto. La mia domanda è: perché è necessario un passaggio manuale, dato che credevo fosse un plugin ufficiale e che avrebbe continuato a funzionare?

[codinghorror (Jeff Atwood)]

È questa una funzionalità che abbiamo spostato dal plugin al core?

In tal caso, dovremmo aggiungere un avviso di deprecazione al plugin… cosa ne pensi @neil?

[elijah (Eli the Bearded)]

Preciso di notare che prima di pubblicare avevo controllato e visto che l’ultimo commit sul plugin era più recente della versione 2.8.0beta2. Rivedendo, noto che il commit proviene da un utente “bot”, quindi potrebbe non essere indicativo di molto.

@codinghorror

È questa una funzionalità che abbiamo spostato dal plugin al core?

Osservando più da vicino, non credo sia esattamente la stessa cosa. Il badge appariva come un’immagine grande nell’angolo in basso a sinistra della scheda utente. Questa nuova funzionalità del core sembra controllare piccole caselle con minuscole immagini di badge e il nome del badge lungo la parte inferiore della scheda utente.

Questo post Why does eviltrout have a garbage man badge? mostra una versione iniziale della funzionalità dei badge:

Come esisteva prima di questo aggiornamento, la selezione era limitata alle immagini associate ai traguardi ottenuti. Ho allegato immagini personalizzate a tutti i badge sul mio forum per rendere più belli i badge sulla scheda utente. (Quelli predefiniti sono, ehm, “spazzatura”.)

[neil (Neil Lalonde)]

Immagino sia la nuova colonna is_favorite aggiunta alla tabella UserBadges.

Il codice SQL è probabilmente rotto ora.

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)]

Ho appena unito una correzione per il plugin:

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

La colonna image è stata rimossa da Badge in UX: Add image uploader widget for uploading badge images (#12377) · discourse/discourse@a23d0f9 · GitHub, e abbiamo aggiornato parzialmente il plugin in quel momento, ma abbiamo tralasciato alcuni punti.

@elijah

La mia domanda è: perché ciò deve essere un passaggio manuale, dato che pensavo fosse un plugin ufficiale e che avrebbe continuato a funzionare.

Sebbene ci impegniamo a evitare che le modifiche al core rompano i plugin, a volte ci sfuggono alcuni utilizzi o i nostri plugin non hanno una copertura dei test sufficiente (eseguiamo i test per tutti i plugin ufficiali ogni volta che viene apportata una modifica al core). Nel commit sopra ho aggiunto più test al plugin discourse-user-card-badges per prevenire problemi simili in futuro.

[elijah (Eli the Bearded)]

@martin

Anche se ci impegniamo a evitare che le modifiche al core rompano i plugin, a volte possiamo non cogliere alcuni utilizzi o i nostri plugin potrebbero non avere una copertura dei test sufficiente (eseguiamo i test per tutti i plugin ufficiali ogni volta che viene apportata una modifica al core).

Capisco che gli errori accadano. Quando ho scritto quello, pensavo che la funzionalità di questo plugin fosse duplicata da una nuova funzionalità del core. Solo dopo aver esplorato ulteriormente tale funzionalità ho realizzato che si tratta di cose diverse.