fix(rgi): correct download flow — confirmOnView + downloadFile by documentPk

The download uses documentPk (not documentTypeId) as the file identifier:
1. confirmOnView/{wid}/{appId}/{documentPk} — confirm view
2. loadDocument/downloadFile/{wid}/{documentPk} — actual download

Removed fileVisibility step (not needed, was causing 404).
Updated page download links to pass documentPk.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
AI Assistant
2026-03-24 21:46:37 +02:00
parent 227c363e13
commit e5e2fabb1d
2 changed files with 12 additions and 40 deletions
+1 -1
View File
@@ -296,7 +296,7 @@ function IssuedDocsPanel({
</div>
<Button size="sm" variant="outline" className="shrink-0 gap-1" asChild>
<a
href={`/api/eterra/rgi/download-doc?workspaceId=${doc.workspaceId || workspaceId}&applicationId=${doc.applicationId || applicationPk}&docId=${doc.documentTypeId}&fileId=${doc.documentPk}`}
href={`/api/eterra/rgi/download-doc?workspaceId=${doc.workspaceId || workspaceId}&applicationId=${doc.applicationId || applicationPk}&documentPk=${doc.documentPk}`}
target="_blank"
rel="noopener noreferrer"
>
+11 -39
View File
@@ -5,33 +5,22 @@ export const runtime = "nodejs";
export const dynamic = "force-dynamic";
/**
* GET /api/eterra/rgi/download-doc?workspaceId=127&applicationId=X&docId=Y&fileId=Z
* GET /api/eterra/rgi/download-doc?workspaceId=127&applicationId=X&documentPk=Y
*
* Downloads an issued document from eTerra RGI.
* 3-step flow:
* 1. fileVisibility/{wid}/{appId}/{docId} — check access, get fileId
* 2. confirmOnView/{wid}/{appId}/{fileId} — confirm view
* 3. loadDocument/downloadFile/{wid}/{fileId} — actual download
*
* If fileId is provided, skips step 1.
* 2-step flow:
* 1. confirmOnView/{wid}/{appId}/{documentPk}
* 2. loadDocument/downloadFile/{wid}/{documentPk}
*/
export async function GET(req: NextRequest) {
try {
const workspaceId = req.nextUrl.searchParams.get("workspaceId");
const applicationId = req.nextUrl.searchParams.get("applicationId");
const docId = req.nextUrl.searchParams.get("docId");
let fileId = req.nextUrl.searchParams.get("fileId");
const documentPk = req.nextUrl.searchParams.get("documentPk");
if (!workspaceId || !applicationId) {
if (!workspaceId || !applicationId || !documentPk) {
return NextResponse.json(
{ error: "workspaceId and applicationId are required" },
{ status: 400 },
);
}
if (!docId && !fileId) {
return NextResponse.json(
{ error: "docId or fileId is required" },
{ error: "workspaceId, applicationId and documentPk are required" },
{ status: 400 },
);
}
@@ -44,35 +33,18 @@ export async function GET(req: NextRequest) {
const client = await EterraClient.create(username, password);
// Step 1: Get fileId from visibility check (if not provided)
if (!fileId && docId) {
const visibility = await client.rgiGet(
`rgi/appdetail/issueddocs/fileVisibility/${workspaceId}/${applicationId}/${docId}`,
);
// Extract fileId from visibility response
if (visibility && typeof visibility === "object") {
const v = visibility as Record<string, unknown>;
fileId = String(
v.fileId ?? v.id ?? v.pk ?? v.documentFileId ?? "",
);
}
if (!fileId) {
return NextResponse.json({ error: "fileId not found in visibility response", visibility }, { status: 404 });
}
}
// Step 2: Confirm on view
// Step 1: Confirm on view
try {
await client.rgiGet(
`rgi/appdetail/issueddocs/confirmOnView/${workspaceId}/${applicationId}/${fileId}`,
`rgi/appdetail/issueddocs/confirmOnView/${workspaceId}/${applicationId}/${documentPk}`,
);
} catch {
// Non-critical — continue to download
}
// Step 3: Download
// Step 2: Download file
const { data, contentType, filename } = await client.rgiDownload(
`rgi/appdetail/loadDocument/downloadFile/${workspaceId}/${fileId}`,
`rgi/appdetail/loadDocument/downloadFile/${workspaceId}/${documentPk}`,
);
return new NextResponse(new Uint8Array(data), {