Misskey: MiAuthのチェックは /api/miauth/{session}/check ではなく /miauth/{session}/check にしたい

[YuzuRyo61]

💡 Summary

MiAuthのチェックURLにPOSTすると404になる

🙂 Expected Behavior

セッションIDが有効かつ認証許可した場合、HTTP 200でjson形式のレスポンスが返ってくる

☹️ Actual Behavior

認証許可してもHTTP 404が出る

📝 Steps to Reproduce

1. miauthのセッションIDを作り、ユーザー認証を行う(iconやcallbackは未指定。)
2. /miauth/{セッションID}/check にPOSTする

📌 Environment

Misskey v12.38.1

[YuzuRyo61]

/api/miauth/{セッションID}/check にPOSTしたら正常に返ってきました。

Misskey APIのドキュメントに問題があるようです。

ユーザーが連携を許可した後、https://misskey.io/miauth/{session}/checkにPOSTリクエストすると、レスポンスとしてアクセストークンを含むJSONが返ります。

もし改善できるのであれば、ドキュメントに揃えていただきたいです。

[tamaina]

404が返ってきておかしいなとは思っていたけど、そういう仕様かと思ってた

[YuzuRyo61]

404が返ってきておかしいなとは思っていたけど、そういう仕様かと思ってた

/miauth/{session}はウェブクライアント側のルーターで処理しているようですが、
/miauth/{session}/checkはサーバー側のルーターで処理しているようなので多分サーバー側のルーターのプレフィックスが影響しているのかもしれません。

[tamaina]

/apiのほうを正規のurlにして/miauth/.../checkに来たら301って感じの対応でどうでしょう？

[mei23]

POSTを301するとGETで飛んじゃいます

[YuzuRyo61]

/apiのほうを正規のurlにして/miauth/.../checkに来たら301って感じの対応でどうでしょう？

とにかく、miauthのURLは統一すべきです。

[tamaina]

api統一でいいか

[tamaina]

ドキュメントを書き換えるだけにする

[YuzuRyo61]

router.post('/miauth/:session/check', async ctx => {
    const token = await AccessTokens.findOne({
        session: ctx.params.session
    });

    if (token && !token.fetched) {
        AccessTokens.update(token.id, {
            fetched: true
        });

        ctx.body = {
            ok: true,
            token: token.token,
            user: await Users.pack(token.userId, null, { detail: true })
        };
    } else {
        ctx.body = {
            ok: false,
        };
    }
});

上記のスクリプトは今 src/server/api/index.ts上に書いてあるみたいですが、これをsrc/server/index.tsに移動すればいいのではないでしょうか。

[syuilo]

/api/miauth/{session}/check ではなく /miauth/{session}/check を意図しているのでそのように修正します

[tamaina]

Note: 経過説明

1. 83ec906ee7c5d9bd42b11129f3f0b3df85287b3c をコミットした
2. 83ec906ee7c5d9bd42b11129f3f0b3df85287b3c だとリモートからの投稿を受信できないことに12.39.0リリース直前に気づく #6444
   
   
   
   • 結局バグを孕んだまま12.39.0がリリースされてしまった
   
   
3. 83ec906ee7c5d9bd42b11129f3f0b3df85287b3c をRevertし、とりあえず/api/...なURLを正規のものとした（12.39.1）