Untitled attachment

Notices where this attachment appears

1. Embed this notice
   tateisu​ :force:​:r_9a: (tateisu@mastodon.juggler.jp)'s status on Tuesday, 26-Sep-2023 06:29:03 JST tateisu​ :force:​:r_9a:

   Mastodon 4.2.0で追加された、管理者向けの通知の元データのURL https://api.joinmastodon.org/update-check 4.2.0 が urgent: true になってるの違和感あるな…。

2. Embed this notice
   zunda (zundan@mastodon.zunda.ninja)'s status on Wednesday, 06-Sep-2023 18:03:30 JST zunda

   in reply to

   config/navigation.rb の7行目に

   n.item :software_updates, safe_join([fa_icon('exclamation-circle fw'), t('admin.critical_update_pending')]), admin_software_updates_path, if: -> { ENV['UPDATE_CHECK_URL'] != '' && current_user.can?(:view_devops) && SoftwareUpdate.urgent_pending? }, html: { class: 'warning' }

   とあって環境変数を直接参照してるのに、

   app/services/software_update_check_service.rb の30行目では

   ENV.fetch('UPDATE_CHECK_URL', 'https://api.joinmastodon.org/update-check')

   とデフォルトの値を設定してるのはちょっとまずそうわね。