(bug) isSSL regex is NOT doing what you think it's doing.

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

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

評価

難易度
1/5
見積もり時間
1時間未満
初心者へのやさしさ
35/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
停滞
技術スタック
javascript, node.js

調査の方向性

lib/http-proxy/common.js の 7 行目から始めて、isSSL がどのように判定されるかを調べます。issue の例を再現し、利用可能であれば既存のテストスイートを実行します。HTTPS と WSS の意図されたプロトコル形式のみをチェックが受け入れ、示されている誤検出を拒否すれば完了です。

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

説明

https://github.com/http-party/node-http-proxy/blob/9b96cd725127a024dabebec6c7ea8c807272223d/lib/http-proxy/common.js#L7

/^https|wss/.test("httpstuff") // true
/^https|wss/.test("cowss") // true

I assume you either way

- /^https|wss/
+ /^https:$|^wss:$/
or
+ /^(?:https|wss):$/
or if you don't mind the creation of a capturing group, it makes it easier to read
+ /^(https|wss):$/
主要言語
JavaScript
スター
14.1k
フォーク
2k
PR マージ指標
30日以内にマージされた PR はありません

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

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

http-party/node-http-proxy のほかの issue

http-party/node-http-proxy の issue をすべて見る

似ている issue

JavaScript の issue をもっと見る

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

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