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.

Sign up for GitHub

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

Jump to bottom

トークン再生成などによりAuthentication Failedになった場合、ログアウトするように #9322

Closed
tamaina opened this issue Dec 13, 2022 · 2 comments · Fixed by #10002
Closed

トークン再生成などによりAuthentication Failedになった場合、ログアウトするように #9322

tamaina opened this issue Dec 13, 2022 · 2 comments · Fixed by #10002
Labels
✨Feature This adds/improves/enhances a feature

Comments

@tamaina
Copy link
Contributor

tamaina commented Dec 13, 2022

Summary

自動的にログアウト or ログアウトを促すメッセージを表示
@tamaina tamaina added the ✨Feature This adds/improves/enhances a feature label Dec 13, 2022
@tamaina
Copy link
Contributor Author

tamaina commented Feb 19, 2023

api/iの応答って、

トークン再生成→403 Authentication failed

アカウント削除（admin, ユーザーによる閉鎖）→

{"error":{"message":"Internal error occurred. Please contact us if the error persists.","code":"INTERNAL_ERROR","id":"5d37dbcb-891e-41ca-a3d6-e690c97775ac","kind":"server","info":{"e":{"message":"Could not find any entity of type \"UserProfile\" matching: {\n    \"where\": {\n        \"userId\": \"9bf3pj3yug\"\n    },\n    \"relations\": [\n        \"user\"\n    ]\n}","code":"EntityNotFoundError","stack":"EntityNotFoundError: Could not find any entity of type \"UserProfile\" matching: {\n    \"where\": {\n        \"userId\": \"9bf3pj3yug\"\n    },\n    \"relations\": [\n        \"user\"\n    ]\n}\n    at /home/aqz/misskey/node_modules/.pnpm/[email protected][email protected][email protected]/node_modules/typeorm/entity-manager/EntityManager.js:604:39\n    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)\n    at async file:///home/aqz/misskey/packages/backend/built/server/api/endpoints/i.js:39:33\n    at async ApiCallService.call (file:///home/aqz/misskey/packages/backend/built/server/api/ApiCallService.js:275:16)"}}}}

みたいになっておかしいなと思ったんだけど、アカウント削除が正常に終了していないだけだったっぽいな

tamaina added a commit to tamaina/misskey that referenced this issue Feb 19, 2023
@tamaina
refreshAccounts
982f501 
Resolve misskey-dev#9322
@tamaina tamaina mentioned this issue Feb 19, 2023
Merged
syuilo pushed a commit that referenced this issue Mar 9, 2023
@tamaina @acid-chicken
enhance: アカウント削除時のクライアントの挙動をいい感じにするなど (#10002)
c75afad 
* refreshAccounts
Resolve #9322

* アカウント管理画面でリストを更新するように

* Update packages/frontend/src/account.ts

Co-authored-by: Acid Chicken (硫酸鶏) <[email protected]>

* ✌️

* クライアント起動時は現在ログインしているアカウントのみリフレッシュする

* clean up

* なんかめっちゃ変えた

* refactor

* refactor

* fix lint

---------

Co-authored-by: Acid Chicken (硫酸鶏) <[email protected]>
@tamaina
Copy link
Contributor Author

tamaina commented Mar 9, 2023 •
edited
Loading

ユーザーを完全に消せていないのではなくクエリキャッシュのせいであることにコードをいじってようやく理解した

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
✨Feature This adds/improves/enhances a feature
Projects
None yet
Milestone
No milestone
1 participant
@tamaina