fix: センシティブな画像をアイコン・バナーに指定できないように #13676
Conversation
github-actions
bot
added
packages/frontend
|
このPRによるapi.jsonの差分 差分はこちら--- base
+++ head
@@ -50351,6 +50351,24 @@
}
}
},
+ "AVATAR_IS_SENSITIVE": {
+ "value": {
+ "error": {
+ "message": "The file specified as an avatar is marked as sensitive.",
+ "code": "AVATAR_IS_SENSITIVE",
+ "id": "71bb5e53-4742-4609-b465-36081e131208"
+ }
+ }
+ },
+ "BANNER_IS_SENSITIVE": {
+ "value": {
+ "error": {
+ "message": "The file specified as a banner is marked as sensitive.",
+ "code": "BANNER_IS_SENSITIVE",
+ "id": "e148b34c-9f33-4300-93e0-7817008fb366"
+ }
+ }
+ },
"NO_SUCH_PAGE": {
"value": {
"error": { |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #13676 +/- ##
============================================
- Coverage 77.84% 65.32% -12.53%
============================================
Files 185 1010 +825
Lines 25600 115498 +89898
Branches 487 5560 +5073
============================================
+ Hits 19929 75452 +55523
- Misses 5664 38604 +32940
- Partials 7 1442 +1435 ☔ View full report in Codecov by Sentry. |
|
/preview |
|
/preview |
|
プレビューどうやって見るんだっけ |
|
/preview |
|
「アイコン画像を変更」などを押しても何も起こらないわね |
たぶん治ったと思われる |
|
/preview |
|
コンフリクト解消 |
|
/preview |
|
（これエラー処理のアラート掲出を別関数に追いやるんじゃなくてos.apiWithDialogにカスタムエラーを指定できるような仕組みにしたほうが良いかも） |
done |
|
/prevoew |
|
/preview |
|
selectFile関数は名前通りファイルを選択させるためのAPIだから、そこに追加でメニューを設定してそれ以外のアクション(アイコン/バナー解除)できるようにするというのは設計上微妙な感じがある |
|
とはいえ良い代替案は思いつかない |
selectFile自体にその機能を設けてるわけではないので別にいい気はする |
この仕様では「デフォルトでメディアをセンシティブ設定にする」を設定している人間が、プロフィールからアイコンをアップロードして変更ができなくなってしまうのだわね。 |
|
たぶんロールで「ファイルにNSFWを常に付与」にされたユーザーは、もうアイコンを変更できなくなってしまう。 |
かといってセンシティブがついている画像を指定できてしまうのもそれはそれで問題だとは思うけど… |
まあ外すことはできるようになったのでいいんじゃないかしら（「ファイルにNSFWを常に付与」を付与される人って基本的に何らかのペナルティを食らっている人のはずなので…？） |
|
コンフリクト解消 |
Co-authored-by: anatawa12 <anatawa12@icloud.com>
What
i/updateでセンシティブな画像の場合はエラーで弾くようにWhy
Fix #7311
Additional info (optional)
Checklist