eCourt-note/app/api/note/files/open/route.ts
Your Name 83a2cc3e5e
All checks were successful
Deploy eCourt Note Web to Server / Validate Database Schema (Drizzle) (push) Successful in 4s
Deploy eCourt Note Web to Server / Build Web Docker Image (push) Successful in 1m10s
Deploy eCourt Note Web to Server / Deploy eCourt Note Web to Server (push) Successful in 4s
Deploy eCourt Note Web to Server / Notify Deployment Status (push) Successful in 1s
fix(note): use relative redirect Location behind proxy
หลัง reverse proxy ที่เขียนทับ Host เป็น upstream (nginx default
\$proxy_host เช่น 0.0.0.0:3000) การ 302 ด้วย NextResponse.redirect(
new URL(path, request.url)) ประกอบ absolute URL จาก origin ฝั่ง server
ทำให้บน prod กดลิงก์เปิดเอกสารตัวอย่างแล้ว browser ถูกพาไป
https://0.0.0.0:3000/docedit/note/annotate/... แทนโฮสต์จริง

เปลี่ยนเป็น Location แบบ relative (NextResponse + header เอง) ให้
browser resolve เทียบ origin ที่ตัวเองยิงมา — ถูกเสมอไม่ว่า proxy
จะตั้ง header แบบไหน:

- GET /api/note/files/open: 302 เข้า viewer และ 302 หน้าแรก
  กรณี fileError
- GET /api/note/auth/dev-link: 302 เข้า /note?fileUrl=…&t=…

ไม่แตะ dev-link POST ที่คืน {url} เต็ม — dev-only และลิงก์ต้อง
absolute เพื่อคัดลอกไปเปิดที่อื่น

Co-Authored-By: Claude Code <noreply@anthropic.com>
2026-11-02 11:12:11 +07:00

43 lines
2.9 KiB
TypeScript

// GET /api/note/files/open?fileUrl=/samples/… — ทางเปิดเอกสารตัวอย่างของหน้า /note/dev
// แบบ "A Link Get(new)" ของ /tmpl (คำขอผู้ใช้ 2026-10-30): ลิงก์เปล่า <Link> ยิง GET ได้เลย
// ไม่มี Bearer (browser navigation ใส่ header ไม่ได้) จึง**รับเฉพาะ path ในเครื่องที่ขึ้นต้น
// "/" เท่านั้น** — registerExternalFile ตรวจว่ามีไฟล์จริงใน public/ ผ่าน path guard เดิม
// (สร้างแถว url:… เท่านั้น — แถว ecourt: เกิดไม่ได้เพราะไม่รับ fileId และ path รีโมตโดนเด็ดขาด)
// แล้ว 302 เข้า viewer · ไฟล์ไม่ผ่าน = 302 /note?fileError=… ให้หน้าแรกแสดงสาเหตุ
import { NextResponse } from "next/server";
import { registerExternalFile } from "@/lib/external-import";
import { withBasePath } from "@/utils/base-path";
export const runtime = "nodejs";
export const dynamic = "force-dynamic";
export async function GET(request: Request): Promise<Response> {
const url = new URL(request.url);
// withBasePath("") = หน้าแรก /docedit/note · withBasePath("/annotate/x") = viewer —
// อย่าส่ง "/note" เข้า withBasePath (มันเติม app path ให้อีกชั้นเป็น /note/note — D11)
// Location เป็น relative โดยเจตนา: browser resolve เทียบ origin ที่ตัวเองยิงมาเอง
// ส่วน NextResponse.redirect(new URL(path, request.url)) จะประกอบ absolute URL จาก
// Host ที่ reverse proxy ส่งมา (nginx default เขียนทับเป็น upstream เช่น 0.0.0.0:3000)
// แล้วพา browser หลุดโฮสต์จริง (prod 2026-11-02)
const redirectTo = (path: string, query = ""): NextResponse => {
return new NextResponse(null, {
status: 302,
headers: { location: `${withBasePath(path)}${query}`, "cache-control": "no-store" },
});
};
const fileUrl = url.searchParams.get("fileUrl") ?? "";
if (!fileUrl.startsWith("/") || fileUrl.includes("..") || fileUrl.length > 2048) {
return redirectTo(
"",
`?fileError=${encodeURIComponent("ลิงก์เปิดเอกสารตัวอย่างต้องเป็น path ในเครื่อง เช่น /samples/…")}`,
);
}
const result = await registerExternalFile(fileUrl, undefined, undefined, { trusted: false });
if (!result.ok) {
return redirectTo("", `?fileError=${encodeURIComponent(result.error)}`);
}
return redirectTo(`/annotate/${result.fileId}`);
}