tasuren / niconico-dl Goto Github PK

初めまして。y-chanといいます。
ニコニコ動画の仕様変更に対応したダウンローダーを探していたところ、こちらにたどり着き、コードを読み、参考にさせていただいています。

さて、本題ですが、PythonにはType Hintsという仕組みがあります。
Type Hintsを使えば動的型付けのPythonでありながら、それなりに型安全なコード(もちろん完全とは言えませんが)を書くことが出来、IDEなどでの補完も充実するようになります。
また、可読性も向上します。
以下は例です

def x_plus_y(x: int, y: int) -> int:
    return x + y

私の方でType Hintsを実装し、PRしようかと思っていますが、必要がなければIssueはcloseします。

初めまして、Akomeと申します！
個人でニコニコ動画を流せるdiscord音楽botを開発していて、そこでこのライブリを使わせていただいたのですが、heartbeatに関して気になるところがいくつかあったのでissue立てさせていただきました！

heartbeatが動いていない

niconico_dl_asyncの方なのですが、コードを追ってみてもheartbeatが動いていると思えませんでした。
実際、get_download_link()で取得したurlも一定時間後に403エラーが帰ってくる状況でした。

heartbeatで一部よくわからない処理がある。

https://github.com/tasuren/niconico_dl/blob/84b8d1e0bbdb353876485c713aa03b33b10370e1/niconico_dl_async/__init__.py#L142-L173
こちらの部分なのですが、151から162行目以外の処理の意味がわかりませんでした。
自分のこちらのForkで僭越ながらいじらせていただいたのですが、151~162行目の処理だけで問題なく動いているようでした。自分が気づけていない意図がありましたら申し訳ないです。

あと、この機能があると素晴らしく使いやすいな...と単純におもったので...
提案させていただきました...