Cookie の節の書き直し by chelproc · Pull Request #860 · ut-code/utcode-learn

chelproc · 2025-08-11T02:22:54Z

No description provided.

cloudflare-workers-and-pages · 2025-08-11T02:25:56Z

Deploying utcode-learn with Cloudflare Pages

Latest commit:	`15e1d16`
Status:	✅ Deploy successful!
Preview URL:	https://33f779b4.utcode-learn.pages.dev
Branch Preview URL:	https://rewrite-cookie.utcode-learn.pages.dev

View logs

Copilot

Pull Request Overview

This PR rewrites the Cookie section of the documentation to improve its structure and provide a more comprehensive guide to authentication in web applications. The changes focus on reorganizing content to better explain cookies in the context of authentication systems.

Key changes include:

Complete restructuring of the Cookie documentation with new authentication-focused content
Addition of new term definitions for HTTP status codes and method terminology updates
Addition of comprehensive sample applications demonstrating cookie-based authentication

Reviewed Changes

Copilot reviewed 16 out of 29 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/components/Term/type-map.js	Adds mapping for HTTP status code term
src/components/Term/definitions.js	Updates HTTP method name and adds status code definition
docs/4-advanced/01-cookie/index.mdx	Complete rewrite focusing on authentication with cookies
docs/4-advanced/01-cookie/_samples/	New sample applications for cookie counter and authentication
docs/3-web-servers/07-fetch-api-post/index.mdx	Updates status code reference to use new term
.vscode/settings.json	Adds Prisma formatter configuration

Files not reviewed (2)

docs/4-advanced/01-cookie/_samples/cookie-counter/package-lock.json: Language not supported
docs/4-advanced/01-cookie/_samples/simple-authentication/package-lock.json: Language not supported

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

docs/4-advanced/01-cookie/_samples/simple-authentication/main.mjs

docs/4-advanced/01-cookie/_samples/simple-authentication/.gitignore

docs/4-advanced/01-cookie/index.mdx

docs/4-advanced/01-cookie/cookie-counter-flow.drawio.svg

docs/4-advanced/01-cookie/index.mdx

chvmvd · 2025-09-26T11:31:54Z

docs/4-advanced/01-cookie/index.mdx

 app.get("/", (request, response) => {
  // Cookieの値は文字列なので数値に変換が必要
-  const count = parseInt(request.cookies.count) || 0;
+  const count = Number.parseInt(request.cookies.count) || 0;


変更前からそうなっているので申し訳ないのですが、このような論理和演算子や論理積演算子の使い方は、「React」の節ではじめて扱うので、この時点では未履修となっています。

utcode-learn/docs/4-advanced/04-react/index.mdx

Line 323 in 2527d7d

このプログラムは、`age`変数が`18`以上である場合のみメッセージを表示します。これは、`&&`演算子の挙動を利用した手法です。これまで、`&&`演算子は両辺が`true`であれば`true`を返す演算子であるとしてきました。しかしながら、[`&&`演算子のより一般的な定義](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Operators/Logical_AND)は、**左辺が[<Term>truthy</Term>](https://developer.mozilla.org/ja/docs/Glossary/Truthy)であれば右辺の値を、そうでなければ左辺の値を返す演算子**です。

docs/4-advanced/01-cookie/simple-authentication-flow.drawio.svg

docs/4-advanced/01-cookie/index.mdx

docs/4-advanced/01-cookie/_samples/simple-authentication/.gitignore

Copilot

Pull Request Overview

Copilot reviewed 17 out of 30 changed files in this pull request and generated 1 comment.

Files not reviewed (2)

docs/4-advanced/01-cookie/_samples/cookie-counter/package-lock.json: Language not supported
docs/4-advanced/01-cookie/_samples/simple-authentication/package-lock.json: Language not supported

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot · 2025-10-05T03:06:27Z

docs/4-advanced/01-cookie/index.mdx

+
+`/login`は、IDとパスワードを含むJSON形式のPOSTリクエストを受け取り、データベースの`User`テーブルのデータと比較することで、認証情報が正しいかどうかを検証します。正しければ、[`crypto.randomUUID`関数](https://developer.mozilla.org/ja/docs/Web/API/Crypto/randomUUID)を用いてランダムな文字列を生成し、新しいレコードを`Session`テーブルに作成した上で、Cookieとしてクライアントに送信します。誤っていれば、直ちに認証失敗の<Term>ステータスコード</Term>を返して終了します。
+
+新しいレコードを`Session`テーブルに作成し、その`sessionId`をCookieとしてクライアントに送信します。誤っていれば、直ちに認証失敗の<Term>ステータスコード</Term>を返して終了します。


This sentence appears to be duplicated content from the previous paragraph (line 114). The first part repeats information already explained, making the documentation redundant.

Suggested change

新しいレコードを`Session`テーブルに作成し、その`sessionId`をCookieとしてクライアントに送信します。誤っていれば、直ちに認証失敗の<Term>ステータスコード</Term>を返して終了します。

chvmvd

ありがとうございます。
後は、ここだけだと思います。

chvmvd · 2025-10-05T03:05:26Z

docs/4-advanced/01-cookie/index.mdx

+`/login`は、IDとパスワードを含むJSON形式のPOSTリクエストを受け取り、データベースの`User`テーブルのデータと比較することで、認証情報が正しいかどうかを検証します。正しければ、[`crypto.randomUUID`関数](https://developer.mozilla.org/ja/docs/Web/API/Crypto/randomUUID)を用いてランダムな文字列を生成し、新しいレコードを`Session`テーブルに作成した上で、Cookieとしてクライアントに送信します。誤っていれば、直ちに認証失敗の<Term>ステータスコード</Term>を返して終了します。
+
+新しいレコードを`Session`テーブルに作成し、その`sessionId`をCookieとしてクライアントに送信します。誤っていれば、直ちに認証失敗の<Term>ステータスコード</Term>を返して終了します。


文章に重複が見られます。

すみません・・・・

chvmvd · 2025-10-05T03:05:46Z

docs/4-advanced/01-cookie/index.mdx

+}
+```
+
+`/login`は、IDとパスワードを含むJSON形式のPOSTリクエストを受け取り、データベースの`User`テーブルのデータと比較することで、認証情報が正しいかどうかを検証します。正しければ、[`crypto.randomUUID`関数](https://developer.mozilla.org/ja/docs/Web/API/Crypto/randomUUID)を用いてランダムな文字列を生成し、新しいレコードを`Session`テーブルに作成した上で、Cookieとしてクライアントに送信します。誤っていれば、直ちに認証失敗の<Term>ステータスコード</Term>を返して終了します。


ここでの「ランダムな文字列」とPrisma Schemaに書かれている「一意でランダムなID」が同じものを指しているということが少しわかりにくいのかなと思いました。
つまり、crypto.randomUUID関数で生成されたランダムな文字列が、「一意」であるということが初学者には非自明ではないかと思いました。
「crypto.randomUUID関数を用いて一意でランダムなIDを生成し」などでもよいのかなと。

確かに！ありがとうございます🙇🙇🙇

chvmvd

ありがとうございます！

Cookie の節の書き直し

2b9479a

chvmvd mentioned this pull request Aug 17, 2025

「Cookieと認証」の節からフォームに関連した記述を削除 #861

Merged

chelproc added 5 commits September 6, 2025 20:45

update

b40a86a

Cookie の利用セクションを更新

206d05b

サーバー側のみ

abd567a

2つめのセクション執筆完了

90a6d13

Merge branch 'main' into rewrite-cookie

4ef158f

chelproc requested a review from chvmvd September 20, 2025 07:05

chelproc marked this pull request as ready for review September 20, 2025 07:05

chvmvd requested a review from Copilot September 26, 2025 14:19

Copilot AI reviewed Sep 26, 2025

View reviewed changes

docs/4-advanced/01-cookie/_samples/simple-authentication/main.mjs Outdated Show resolved Hide resolved

docs/4-advanced/01-cookie/_samples/simple-authentication/.gitignore Show resolved Hide resolved

chvmvd reviewed Sep 26, 2025

View reviewed changes

chvmvd mentioned this pull request Sep 28, 2025

2025-10のリリース #880

Closed

7 tasks

chelproc added 2 commits October 4, 2025 19:44

レビュー修正

57c5bff

フォントの問題を修正

d7290fd

chvmvd requested a review from Copilot October 5, 2025 03:06

Copilot AI reviewed Oct 5, 2025

View reviewed changes

chvmvd reviewed Oct 5, 2025

View reviewed changes

レビュー修正

15e1d16

chelproc requested a review from chvmvd October 13, 2025 12:19

chvmvd approved these changes Oct 13, 2025

View reviewed changes

chvmvd merged commit cceb0dd into main Oct 13, 2025
3 checks passed

chvmvd deleted the rewrite-cookie branch October 13, 2025 12:54

chvmvd mentioned this pull request Dec 13, 2025

チャットアプリと掲示板のステータスコードを修正 #902

Merged


		`/login`は、IDとパスワードを含むJSON形式のPOSTリクエストを受け取り、データベースの`User`テーブルのデータと比較することで、認証情報が正しいかどうかを検証します。正しければ、[`crypto.randomUUID`関数](https://developer.mozilla.org/ja/docs/Web/API/Crypto/randomUUID)を用いてランダムな文字列を生成し、新しいレコードを`Session`テーブルに作成した上で、Cookieとしてクライアントに送信します。誤っていれば、直ちに認証失敗の<Term>ステータスコード</Term>を返して終了します。

		新しいレコードを`Session`テーブルに作成し、その`sessionId`をCookieとしてクライアントに送信します。誤っていれば、直ちに認証失敗の<Term>ステータスコード</Term>を返して終了します。

Conversation

chelproc commented Aug 11, 2025

Uh oh!

cloudflare-workers-and-pages bot commented Aug 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploying utcode-learn with Cloudflare Pages

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

chvmvd Sep 26, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI Oct 5, 2025

Choose a reason for hiding this comment

Uh oh!

chvmvd left a comment

Choose a reason for hiding this comment

Uh oh!

chvmvd Oct 5, 2025

Choose a reason for hiding this comment

Uh oh!

chelproc Oct 13, 2025

Choose a reason for hiding this comment

Uh oh!

chvmvd Oct 5, 2025

Choose a reason for hiding this comment

Uh oh!

chelproc Oct 13, 2025

Choose a reason for hiding this comment

Uh oh!

chvmvd left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

cloudflare-workers-and-pages bot commented Aug 11, 2025 •

edited

Loading