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

Toot IDに対する制約は間違いなのではないか #90

Closed
yumetodo opened this issue Jan 13, 2020 · 11 comments · Fixed by #72
Closed

Toot IDに対する制約は間違いなのではないか #90

yumetodo opened this issue Jan 13, 2020 · 11 comments · Fixed by #72

Comments

@yumetodo
Copy link
Contributor

from #72

現在、Toot IDについて「1.0+E18より小さければ、id の途中で url が切れたと判断してエラーに」という仕様があるが、これは誤りではないか。

反例: https://qiitadon.com/@tomoasleep/1

@sasanquaneuf
Copy link
Contributor

そうですね、他のidのトゥートが引っかかってるのかなと思いましたが、そうではなくて本当にid=1のトゥートが存在していますね。

@yume-yu
Copy link
Collaborator

yume-yu commented Jan 13, 2020

これは #88 の実装にも関連しそうなので気になります:eyes:

@yumetodo
Copy link
Contributor Author

https://qithub-bot.github.io/mastogetter/p.html?i=https://qiitadon.com&t=4901,4912,4987,4989,4990,4991,5000,5211,5555

これを見てみると全部ってわけではないけど小さい数字のが見受けられます。

多分MastdonのTooT ID生成にどっかのタイミング(v1.3付近?)でbreaking chnageが入ってると思われます。証拠を見つけてないですが。

@yumetodo
Copy link
Contributor Author

id途切れを捕捉する事自体をやめるべきだと思います。どうしてもid途切れを捕捉するならid listのhashかなにかを取るべきではないでしょうか。

@yume-yu
Copy link
Collaborator

yume-yu commented Jan 13, 2020

urlへの対応はラフに実装した部分だったと思うので、必要ならば変えてしまっておっけーだと思います

@yumetodo
Copy link
Contributor Author

955beed

このcommitですね。
cc: @hidao80

@KEINOS
Copy link
Member

KEINOS commented Jan 13, 2020

数桁の ID があるのは、私も以前自分のトゥート全消しをスクリプトでした際に気づきました。

ある時点からトゥートの ID の振り方が大幅に(桁数が)変わっています。創造主のオイゲンさんがそういう仕様にしたので、仕様がない。おそらく Mastodon が Activity Pub 対応の準備をはじめたあたりかと。

数桁の PubSubHubBabu(WebSub?) 時代は、おいどんも知らない前世代なので遺跡を掘る感じで面白いですね。1.0+E18 は、1つの時代の区切り線なのかもしれません。

https://qithub-bot.github.io/mastogetter/p.html?i=https://qiitadon.com&t=1,2,3,4,5

遺跡トゥートも API で拾えてきてるし、私も必要なら変えてしまっておkだと思いまーす。

@yumetodo
Copy link
Contributor Author

では変える方向で #72 で実装します。

yumetodo added a commit to yumetodo/mastogetter that referenced this issue Jan 13, 2020
@yumetodo
Copy link
Contributor Author

実装しました

@KEINOS
Copy link
Member

KEINOS commented Jan 13, 2020

はやっ!

@yumetodo
Copy link
Contributor Author

もともとこれのために冗長なコードが必要になってたのでついでにスッキリ。

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.

4 participants