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

request.routerPathが非推奨になった影響で、警告 [FSTDEP017] が表示される (refactor) #11934

Closed
AsPulse opened this issue Sep 30, 2023 · 1 comment · Fixed by #11935

Comments

@AsPulse
Copy link
Contributor

AsPulse commented Sep 30, 2023

💡 Summary

misskeyを立ち上げたときに、下記の警告が生成されます。


(node:229) [FSTDEP017] FastifyDeprecation: You are accessing the deprecated "request.routerPath" property. 
Use "request.routeOptions.url" instead. Property "req.routerPath" will be removed in fastify@5.

Description

エラー文の通り、request.routerPathfastify@5で消えるので新しいのを使ってねということみたいです。(今misskeyで使われているのは最新の4.23.2)
使われているのはこの一箇所しか見つからないのですが、これが全リクエストごとに走っているので修正したいです!

// Authenticate
fastify.addHook('onRequest', async (request, reply) => {
// %71ueueとかでリクエストされたら困るため
const url = decodeURI(request.routerPath);
if (url === bullBoardPath || url.startsWith(bullBoardPath + '/')) {


適当なbackendのテスト (例えばこことか) の「Test」ステップを覗いてみると、このWarningが大量に生成されていることが分かると思います。
image

@AsPulse AsPulse changed the title requrest.routerPathが非推奨になった影響で、警告 [FSTDEP017] が表示される (refactor) request.routerPathが非推奨になった影響で、警告 [FSTDEP017] が表示される (refactor) Sep 30, 2023
@AsPulse
Copy link
Contributor Author

AsPulse commented Sep 30, 2023

PR作成させていただきました! (自分をIssueにアサインしろってCONTRIBUTING.mdにあったんですが権限がなく……どなたかお願いします……)

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