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.

Sign up for GitHub

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

Jump to bottom

バイナリのWebSocketデータのハンドリングに対応する #87

Closed
3 tasks done
MtkN1 opened this issue Oct 12, 2021 · 0 comments
Closed
3 tasks done

バイナリのWebSocketデータのハンドリングに対応する #87

MtkN1 opened this issue Oct 12, 2021 · 0 comments
Assignees
@MtkN1
Labels
enhancement New feature or request

Comments

@MtkN1
Copy link
Member

MtkN1 commented Oct 12, 2021 •
edited
Loading

課題

既存の対応取引所においてはバイナリデータをWebSocketで送受信するケースはないが、他の取引所においてはそのケースが存在する。

Huobi Global

https://huobiapi.github.io/docs/spot/v1/en/#websocket-market-data

この場合はpybottersは aiohttp.WSMsgType.TEXT しか受け付けておらず、ws_connectの引数もstr及びjsonのみであるので全くメッセージをハンドリングできない。
aiohttpでは元々bytesのメソッドも存在しているので、既存取引所のケースにはなかったがpybottersのWebSocketアプリケーション役割としてはやはり追加するのが望ましい。

Discordより

https://discord.com/channels/832651305155297331/832653694839160892/893672780464291850

実装

aiohttp.WSMsgType.BINARY のハンドリングを追加し、ws_connectの引数にsend_bytes及びhdlr_bytesを追加する。

チェックリスト

  • 実装
  • テスト
  • Wiki
@MtkN1 MtkN1 added the enhancement New feature or request label Oct 12, 2021
@MtkN1 MtkN1 self-assigned this Oct 12, 2021
MtkN1 added a commit that referenced this issue Nov 10, 2021
@MtkN1
Add binary data send/handle to websocket (#87)
192735a
@MtkN1 MtkN1 mentioned this issue Nov 11, 2021
Merged
MtkN1 added a commit that referenced this issue Nov 11, 2021
@MtkN1
Merge pull request #94 from MtkN1/develop
ec08e33 
✨v0.8.0リリース

## Issues
✅ Bybitの型問題の抜本的な解決方法について #82
✅ バイナリのWebSocketデータのハンドリングに対応する #87
✅ GMOコインのデータストアにタイムスタンプを追加する #88
## Pull requests
✅ Remove default empty list #92
✅ add API Reference #93
@MtkN1 MtkN1 closed this as completed Jan 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MtkN1 MtkN1

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@MtkN1