getappmap / getappmap/appmap-python

django integration bypassed when config is missing

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

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

bug
主要言語
Python
スター
104
フォーク
17
PR マージ指標
30日以内にマージされた PR はありません

説明

When APPMAP=true, but the config file is missing, the django middleware forwards all requests to the app. This means that requests for /_appmap/record will fail, without giving a clear indication of the problem.

It would be better if the requests for /_appmap/record failed with an error (e.g. 500), to help the user understand the problem

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

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

はじめの一歩

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

調査の方向性

APPMap と /_appmap/record リクエストを処理する Django ミドルウェアを特定し、設定ファイルなしで APPMAP=true にした場合の動作を再現してください。完了条件は、/ _appmap/record へのリクエストがアプリケーションに転送されるのではなく、明確なエラーを返すことです。

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

評価

技術スタック
django, python
領域
backend
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

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

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