Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

禁止ワード、及びPR149,150の内容によってフィルタされた投稿はDelayedキューに溜まってしまう #151

Closed
Sayamame-beans opened this issue Feb 21, 2024 · 8 comments · Fixed by #153

Comments

@Sayamame-beans
Copy link

💡 概要

タイトル通りです。
弾いたならDelayedじゃなくて消えるべきです。

前者は本家にも、後者はうちだけにパッチが必要

🥰 想定される挙動

Delayedしない

🤬 Actual Behavior

Delayedする

📝 再現手順

No response

💻 お使いの環境

* 機種とOS:
* ブラウザ:
@anatawa12
Copy link
Collaborator

failedじゃなくてdelayedにいくのか

@Sayamame-beans
Copy link
Author

そう
Errorだからかもしれない

@Sayamame-beans
Copy link
Author

Sayamame-beans commented Feb 21, 2024

throw new IdentifiableError('e11b3a16-f543-4885-8eb1-66cad131dbfd', 'Notes including mentions, replies, or renotes from remote users are not allowed until user has at least one local follower.');

@anatawa12
Copy link
Collaborator

bullmq側に根拠を求めてました。見つかった。 https://docs.bullmq.io/guide/retrying-failing-jobs

@anatawa12
Copy link
Collaborator

NoteCreateService.ContainsProhibitedWordsError、IdentifiableErrorにしたいな

@anatawa12
Copy link
Collaborator

upstreamに投げる前提でそうしちゃうか

@anatawa12
Copy link
Collaborator

後者、多分IOにも投げるべきだな。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants