No Pusher method/options to add Http headers for web-socket connection establishment
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 42/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- java
- 領域
- networking
調査の方向性
Start in src/main/java/com/pusher/client/connection/websocket/WebSocketClientWrapper.java, where the superclass constructor call is made, and in PusherOptions, which has no way to set headers today. The issue proposes a constructor that accepts a header map and forwards it to the superclass, plus a PusherOptions setter that passes the map down the chain. Done means an origin header set through PusherOptions reaches the websocket handshake. The issue has no comments or linked pull request, so check with maintainers about the API shape before writing code.
索引モデルが issue の本文から書いたものです。
説明
What is the issue?
I need to send "origin" header to the server while I establish connection to it. But I couldn’t find any method or pusher options to do so. And the httpAuthorizer only sends headers when a private subscription is performed. As I investigated the Webscoket implementation for this library, I see that there is a WebsocketClient Constructor which takes in httpHeaders as an optional parameter which can solve my issue.
Any improvements you suggest
An additional constructor can be added to WebSocketClientWrapper (which extends the WebsocketClient) which takes a map of httpHeaders as a parameter and then update the super call to send in the httpHeaders.
With this change, a new PusherOptions method can be added which can take httpHeaders as a parameter and supply it up the chain.
CC @pusher/mobile
- 主要言語
- Java
- スター
- 315
- フォーク
- 140
- 平均マージ
- 10日 14時間
- マージ済み PR（30日）
- 5
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
pusher/pusher-websocket-java のほかの issue
-
java.lang.NoClassDefFoundError: com.pusher.client.Pusher$$ExternalSyntheticLambda0 on Android API 21-22 anyway対応中かも @Keith-wright が 12 日前に担当しました。 オープン
難易度 4/5 3〜5日 初心者へのやさしさ 22/100
pusher/pusher-websocket-java#366 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 15/100
pusher/pusher-websocket-java#340 · コメント 3 件 · リアクション 7 件 ·
-
難易度 4/5 1〜2日 初心者へのやさしさ 8/100
pusher/pusher-websocket-java#311 · コメント 4 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 12/100
pusher/pusher-websocket-java#308 · コメント 2 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 18/100
pusher/pusher-websocket-java#303 · コメント 4 件 ·
pusher/pusher-websocket-java の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 3 日以内に返信
-
A proxy that fails to start logs a RejectedExecutionException stack trace from its shutdown hookオープンfollow-up
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
Sanne/incus-spawn#1953 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
Shweit/EtherealPortals#30 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 70/100
checkstyle/checkstyle#22398 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Typo in jakartaee-bom 12.0.0-SNAPSHOT with cdi対応中かも @ElectLive が今日担当しました。 オープン
難易度 1/5 1時間未満 初心者へのやさしさ 80/100
jakartaee/jakartaee-api#205 · コメント 1 件 ·