heartbeatが動いていない about niconico-dl

[tasuren]

コメントとPRありがとうございます。
ずっと前から忙しく放置してました...
了解です。時間があれば詳しく見てみたいと思います。
そしてまだまだPythonが未熟なころに作ったものなのでいつかコードを書き直したいと思います。

151~162の箇所はニコニコ動画の通信の仕様上必要な箇所です。
ニコニコ動画から動画を取得する際は定期的にhttps://api.dmc.nico/api/sessions/{session_id}?_format=json&_method=PUTに動画データを送らなければ通信が切れてしまいます。
これをしている箇所です。
(niconico_dlを作ろうと思った時の仕様で今の仕様もこの仕様かは不明です。)

[akomekagome]

分かりにくいissueで申し訳ないです！
おっしゃる通り151~162行目の部分は必要な部分だと思いますが、それ以外の部分の意味が分からなかったため質問させていただいたきました。

自分の音楽botも151~162行目の処理だけで動いているので、特に仕様の変更はないと思います！

[y-chan]

async版のheartbeatに関してですが、まだ私自身も試したわけではないのでうまくいく確証はありませんが、asyncio.gatherをうまく使えば、heartbeatを動かしながらプログラム内でのダウンロードができると思います(現状のコードは、heartbeatを動かしてクライアントが生きていることを示し続ける代わりに手動ダウンロードする形になっているという認識ですがあってますかね...?)。

[tasuren]

async版のheartbeatに関してですが、まだ私自身も試したわけではないのでうまくいく確証はありませんが、asyncio.gatherをうまく使えば、heartbeatを動かしながらプログラム内でのダウンロードができると思います(現状のコードは、heartbeatを動かしてクライアントが生きていることを示し続ける代わりに手動ダウンロードする形になっているという認識ですがあってますかね...?)。

@y-chan
すみません返信するの忘れてました。
そしてまたすみません書き直す前の状態がどんなのか忘れたのですぐには返信できません。
書き直し後はheartbeatをloop.create_taskで動かしています。

@akomekagome
コードを書き直して動くことを確認したので恐らく動くと思います。

[tasuren]

今日リリースしたバージョンでうまく動くのを確認したのでcloseします。