Doneの定義

• PR作成時にテスト、Linterが実行されている事

Doneの定義

• 表題の件が実施されている事

Doneの定義

• issue, PRのTemplateが追加されている事
• Cloudflare Workers に簡単なAPIが作成されデプロイが行われている事
• READMEに環境構築の方法が記載されている事

概要

表題の通り。

再現手順

最新のステージング環境に対してPOST /cat-images/validation-results を行うとCORSのエラーが返ってくる。

何故かGETのリクエストは通る。

再現環境

最新のステージング環境。

このバグによって引き起こされる問題

BFFを利用してPOSTリクエストを送信出来ないので早めに解決を行う。

スクリーンショット、エラーメッセージ

Access to fetch at 'http://localhost:8787/lgtm-images' from origin 'http://localhost:2222' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: It does not have HTTP ok status.

補足情報

特になし

Done の定義

• クライアントクレデンシャルでアクセストークンを発行するエンドポイントが実装されている事

補足情報

アクセストークン発行用エンドポイントの実装を行う事でcacheを利用出来るようにする。

Doneの定義

• GET /lgtm-images へ通信を行う為のエンドポイントが実装されている事

Doneの定義

• 404時のカスタムレスポンスが設定されている事
• 例外がThrowされた際に専用のエラーレスポンスを返すようになっている事
• 例外がThrowされた際にSentryに通知が行われるようになっている事

Done の定義

• 定義されているカスタムErrorの書き方が以下のFormatに変更されている事

class MyError extends Error {
  static {
    this.prototype.name = "MyError";
  }
}

補足情報

https://www.wantedly.com/companies/wantedly/post_articles/492456

• this.name が変化するので name は静的な値を定義する
• Error.captureStackTrace(this, MyError); のような記述は現代では不要

Doneの定義

• 表題のAPIへの通信ルートが追加されている事

Done の定義

• 各APIのE2Eテストが実装されている事

補足情報

• https://github.com/honojs/hono-minimal

• https://blog.cloudflare.com/ja-jp/miniflare-ja-jp/

Done の定義

• READMEを見て開発環境の構築が出来るようになっている事
• 環境変数の登録方法がREADMEに記載されている事

補足情報

特になし

Done の定義

• 表題の件が実行されている事

補足情報

https://github.com/honojs/hono/releases/tag/v3.0.0

Doneの定義

• ESLint, Prettierの設定と関連Packageが追加されている事

Done の定義

• prettier-plugin-sort-imports が導入されている事

補足情報

eslint-plugin-import は場合によっては削除する。

Doneの定義

• main ブランチにマージした時にステージング環境にデプロイされるようになっている事
• リリースページ作成時に本番環境にデプロイされるようになっている事

Doneの定義

• 表題の内容のエンドポイントが実装されている事

Done の定義

• HTTPHeaderに Access-Control-Allow-Origin が追加されている事
• CORSでのRequestが正常に動作するようになっている事

補足情報

CORSリクエストが動作しないので、早めに対応を行う。