Fixes `path.resolve(false)` in `MIDDLEWARE_PATH` declaration.
まだ誰も着手していません。
評価
- 難易度
- 1/5
- 見積もり時間
- 1時間未満
- 初心者へのやさしさ
- 50/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 停滞
- 技術スタック
- express, javascript, node.js
- 領域
- backend
調査の方向性
index.js の MIDDLEWARE_PATH 宣言から始め、process.argv[3] の現在の処理を issue に提示された diff と比較します。可能であれば backstop のリモート middleware のケースを再現し、その後、middleware path を指定せずに起動しても引き続き動作すること、また指定された path が false を path.resolve に渡さずに解決されることを確認します。
索引モデルが issue の本文から書いたものです。
説明
Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch [email protected] for the project I'm working on.
While debugging backstop remote I came across an issue with the middleware arg here. path.resolve doesn't accept booleans, so this splits the declaration of MIDDLEWARE_PATH into a ternary instead.
Hope this helps. Thanks for all you do!
Here is the diff that solved my problem:
diff --git a/node_modules/super-simple-web-server/index.js b/node_modules/super-simple-web-server/index.js
index 1eb8065..a27c9b9 100644
--- a/node_modules/super-simple-web-server/index.js
+++ b/node_modules/super-simple-web-server/index.js
@@ -34,7 +34,7 @@ function logger(req, res, next) {
* To add middleware set the third arg to a path that resolves to a file exporting a function like so...
* module.exports = (app) => { app.use(someCoolThingHere);}
*/
-const MIDDLEWARE_PATH = path.resolve(process.argv[3] || false);
+const MIDDLEWARE_PATH = process.argv[3] ? path.resolve(process.argv[3]) : false;
if (MIDDLEWARE_PATH) {
try {
var initMiddleware = require(MIDDLEWARE_PATH);
This issue body was partially generated by patch-package.
- 主要言語
- JavaScript
- スター
- 0
- フォーク
- 2
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
似ている issue
-
status: ready
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
melodic-software/claude-code-plugins#15008 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
S-Needs triage
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 2 日以内に返信
-
ACK_WAITING HELP_WANTED UPDATE_CS
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
OWASP/CheatSheetSeries#2935 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
docs-external-links-failure
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
ComposioHQ/composio#5297 ·
メンテナーはふだん 1 日以内に返信