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

Threadsのリソースurlが長すぎて入らない #70

Closed
slofp opened this issue Jul 18, 2024 · 16 comments · Fixed by #73
Closed

Threadsのリソースurlが長すぎて入らない #70

slofp opened this issue Jul 18, 2024 · 16 comments · Fixed by #73
Assignees
Labels

Comments

@slofp
Copy link
Collaborator

slofp commented Jul 18, 2024

概要

misskeyのDriveFileテーブルの定義ではurlなどの長さはvarchar(512)まで対応していますが、
threadsの投稿にあるリソースurlは画像系はかろうじてvarchar(512)までに収まるものの
動画系に関してはvarchar(512)を超えてしまいテーブル挿入でエラーが起きています。

@slofp slofp added the Backend label Jul 18, 2024
@slofp slofp self-assigned this Jul 22, 2024
@tai-cha
Copy link

tai-cha commented Jul 22, 2024

これ普通にupstreamでやったほうがよさそうな気がする

@tai-cha
Copy link

tai-cha commented Jul 22, 2024

テーブル拡張すればええんよね(とはいえどれぐらい長いの飛んでくるんだろ)

@slofp
Copy link
Collaborator Author

slofp commented Jul 22, 2024

これ普通にupstreamでやったほうがよさそうな気がする

それはそう

テーブル拡張すればええんよね(とはいえどれぐらい長いの飛んでくるんだろ)

そこまでap自体のオブジェクト見たわけじゃないからわからないけどこんなの

https://scontent.cdninstagram.com/o1/v/t16/f2/m69/An-pWZS_M29inL40DPA2DnTJMPMmm_ko-dtmT71ZOhA-cvfrzUn7L8nwSnpvoDZxt1j0YGVWCrFf_9c0j0oehQ.mp4?efg=eyJxZV9ncm91cHMiOiJbXCJpZ193ZWJfZGVsaXZlcnlfdnRzX290ZlwiXSIsInZlbmNvZGVfdGFnIjoidnRzX3ZvZF91cmxnZW4uZmVlZC5jMi4xMjgwLmJhc2VsaW5lIn0&_nc_ht=scontent.cdninstagram.com&_nc_cat=106&vs=1226976408460263_3567848099&_nc_vs=HBksFQIYOnBhc3N0aHJvdWdoX2V2ZXJzdG9yZS9HRUJtNGhyakdMWW5vbUFCQUFFZmFTN1U4WllTYmtZTEFBQUYVAALIAQAVAhg6cGFzc3Rocm91Z2hfZXZlcnN0b3JlL0dENHM1UnA1NnNEZUY2RUJBRjlGU1ZLSldMQlJia1lMQUFBRhUCAsgBACgAGAAbABUAACaUjtTqncnJPxUCKAJDMywXQEesan752yMYEmRhc2hfYmFzZWxpbmVfMV92MREAdeoHAA%3D%3D&_nc_rid=a76d48a21c&ccb=9-4&oh=00_AYAzbtm3ahQctt_A3kshDt4iXRXSf6gViBZO16H3uicN4A&oe=66A07277&_nc_sid=10d13b

@tai-cha
Copy link

tai-cha commented Jul 22, 2024

UpstreamでIssueなかったら作る(こっち(ねむすぎー)で作業しちゃってもいいけどUpstreamにチェリーピックしてPRするかも)

@tai-cha
Copy link

tai-cha commented Jul 22, 2024

自分で動画上げて検証すればいいのか

@slofp
Copy link
Collaborator Author

slofp commented Jul 22, 2024

一応今直しているのでこっち側では普通に治すかも、チェリーピックはまあ任せる

@tai-cha
Copy link

tai-cha commented Jul 22, 2024

これ普通にupstreamでやったほうがよさそうな気がする

それはそう

テーブル拡張すればええんよね(とはいえどれぐらい長いの飛んでくるんだろ)

そこまでap自体のオブジェクト見たわけじゃないからわからないけどこんなの

https://scontent.cdninstagram.com/o1/v/t16/f2/m69/An-pWZS_M29inL40DPA2DnTJMPMmm_ko-dtmT71ZOhA-cvfrzUn7L8nwSnpvoDZxt1j0YGVWCrFf_9c0j0oehQ.mp4?efg=eyJxZV9ncm91cHMiOiJbXCJpZ193ZWJfZGVsaXZlcnlfdnRzX290ZlwiXSIsInZlbmNvZGVfdGFnIjoidnRzX3ZvZF91cmxnZW4uZmVlZC5jMi4xMjgwLmJhc2VsaW5lIn0&_nc_ht=scontent.cdninstagram.com&_nc_cat=106&vs=1226976408460263_3567848099&_nc_vs=HBksFQIYOnBhc3N0aHJvdWdoX2V2ZXJzdG9yZS9HRUJtNGhyakdMWW5vbUFCQUFFZmFTN1U4WllTYmtZTEFBQUYVAALIAQAVAhg6cGFzc3Rocm91Z2hfZXZlcnN0b3JlL0dENHM1UnA1NnNEZUY2RUJBRjlGU1ZLSldMQlJia1lMQUFBRhUCAsgBACgAGAAbABUAACaUjtTqncnJPxUCKAJDMywXQEesan752yMYEmRhc2hfYmFzZWxpbmVfMV92MREAdeoHAA%3D%3D&_nc_rid=a76d48a21c&ccb=9-4&oh=00_AYAzbtm3ahQctt_A3kshDt4iXRXSf6gViBZO16H3uicN4A&oe=66A07277&_nc_sid=10d13b

クエリ長すぎて笑っちゃった

@tai-cha
Copy link

tai-cha commented Jul 22, 2024

確かに動画付きの投稿に照会かけてみたらログめっちゃ長かった

@tai-cha
Copy link

tai-cha commented Jul 22, 2024

一応upstreamにもissue化

misskey-dev#14292

@tai-cha
Copy link

tai-cha commented Jul 22, 2024

これログ見て思ったんだけど長いのURLだけじゃないかもしれない

@slofp
Copy link
Collaborator Author

slofp commented Jul 23, 2024

urlだけじゃない...?
とりあえずDriveFileにあるurl, uri, srcは長過ぎることが確定してるので直してたけど
修正コミット -> 652eaff

@tai-cha
Copy link

tai-cha commented Jul 23, 2024

あーやっぱりuri, srcもだよね thx

@tai-cha
Copy link

tai-cha commented Jul 23, 2024

(これごくまれにThreadsじゃなくて純アップロードでも引っかかるらしい)

@slofp
Copy link
Collaborator Author

slofp commented Jul 23, 2024

(これごくまれにThreadsじゃなくて純アップロードでも引っかかるらしい)

えぇ... (

一応、後でdreamのprに書くけどここにも書いておく、

  • 多分1024で事足りる。流石に長過ぎてもあれだしthreadsが例外的なだけだと思ってる
  • 一度 @tai-cha さんもマイグレーションのクエリ周りを見といてほしい
    • というのもマイグレーションで生成したクエリだとalter tableでカラム削除したあとに追加するようなクエリだったので、本番環境で死ぬと思ってalter table部分をalter columnに手動で差し替えてる

@tai-cha
Copy link

tai-cha commented Jul 23, 2024

実は自動生成知らなくて今まで全部手動で書いてた()

@tai-cha
Copy link

tai-cha commented Jul 23, 2024

ごくまれに

misskey-dev#14216

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

Successfully merging a pull request may close this issue.

2 participants