microsoft / microsoft/playwright-java

[Feature]: allow to define multiple times the same header in route.fulfill

オープン
#1,797 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

P3-collecting-feedback
主要言語
Java
スター
1.6k
フォーク
298
平均マージ
3日 2時間
マージ済み PR(30日)
14

説明

🚀 Feature Request

There is currently no way to define multiple times the same header in route.fulfill.

I first hoped I could trick it by using different cases but com/microsoft/playwright/impl/RouteImpl.java:214 uses headers.put(h.getKey().toLowerCase(), h.getValue());

I would expect that I can define headers in both ways, via Map<String, String> or via List<HttpHeader> (likewise one can use response.allHeaders() and response.headersArray()

Example

I have a test where I want to make sure that my implementation to detect multiple headers work. That's why I use route.fulfill and would like to define a header multiple times.

Motivation

see above

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

com/microsoft/playwright/impl/RouteImpl.java の214行目から始め、route.fulfill が Map<String, String> と List をどのように処理するかを追跡します。response.allHeaders() および response.headersArray() と動作を比較します。route.fulfill が要求された両方の入力形式で重複するヘッダー名を保持できれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java
領域
api, testing
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。