Compare commits
2 Commits
db8dce8c8b
...
28b2edf2a3
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
28b2edf2a3 | ||
|
|
7a67a2febc |
@@ -1,6 +1,7 @@
|
||||
import hashlib
|
||||
from pathlib import Path
|
||||
from typing import Any
|
||||
from urllib.parse import quote_plus
|
||||
|
||||
from fastapi import APIRouter, File, Form, HTTPException, Request, UploadFile
|
||||
from fastapi.responses import RedirectResponse
|
||||
@@ -40,6 +41,11 @@ def _confirm_submission_to_paperless(current: dict) -> dict:
|
||||
if value:
|
||||
fields[optional_key] = value
|
||||
artifact_kind = handoff.get("artifact_kind") or "document"
|
||||
response: dict[str, Any]
|
||||
body_obj: dict[str, Any]
|
||||
ok = False
|
||||
error_detail = None
|
||||
try:
|
||||
if artifact_kind == "source_bundle":
|
||||
paperless_response = paperless.upload_source_bundle_receipt(current)
|
||||
enrichment = paperless.enrich_direct_submission(current, paperless_response)
|
||||
@@ -67,6 +73,17 @@ def _confirm_submission_to_paperless(current: dict) -> dict:
|
||||
body = response.get("body", {}) if isinstance(response, dict) else {}
|
||||
body_obj = body[0] if isinstance(body, list) and body and isinstance(body[0], dict) else (body if isinstance(body, dict) else {})
|
||||
ok = bool(response.get("ok", False)) if isinstance(response, dict) else False
|
||||
if not ok:
|
||||
error_detail = str(response.get("error") or response.get("body") or "Paperless confirmation failed.")
|
||||
except Exception as exc:
|
||||
error_detail = f"{type(exc).__name__}: {exc}"
|
||||
response = {
|
||||
"ok": False,
|
||||
"status_code": 500,
|
||||
"mode": "direct-paperless" if artifact_kind == "source_bundle" else "webhook",
|
||||
"error": error_detail,
|
||||
}
|
||||
body_obj = {}
|
||||
updates = {
|
||||
"status": "paperless-queued" if ok else "paperless-error",
|
||||
"confirmed_at": store.now(),
|
||||
@@ -76,6 +93,7 @@ def _confirm_submission_to_paperless(current: dict) -> dict:
|
||||
"paperless_task_id": body_obj.get("paperless_task_id"),
|
||||
}
|
||||
updated = repo.update_submission(str(current.get("id")), updates) or current
|
||||
updated["paperless_error_detail"] = error_detail
|
||||
grade_note = None
|
||||
if ok and updated.get("assignment_id"):
|
||||
grade_note = paperless.append_grade_note_if_available(str(updated.get("assignment_id")))
|
||||
@@ -87,6 +105,7 @@ def _confirm_submission_to_paperless(current: dict) -> dict:
|
||||
"webhook": [response],
|
||||
"grade_note": grade_note,
|
||||
"file_count": handoff.get("file_count") or 1,
|
||||
"error_detail": error_detail,
|
||||
}
|
||||
|
||||
|
||||
@@ -220,7 +239,10 @@ async def intake_assignment(
|
||||
"backend": repo.backend,
|
||||
}
|
||||
if "text/html" in accept:
|
||||
redirect_url = "/assignments/upload?submitted=" + str(created["id"]) + "&file_count=" + str(bundle["file_count"]) + "&artifact_kind=" + str(created.get("artifact_kind", "document")) + "&paperless_status=" + str(created.get("status", "uploaded-local"))
|
||||
status_value = str(created.get("status", "uploaded-local"))
|
||||
redirect_url = "/assignments/upload?submitted=" + str(created["id"]) + "&file_count=" + str(bundle["file_count"]) + "&artifact_kind=" + str(created.get("artifact_kind", "document")) + "&paperless_status=" + status_value
|
||||
if auto_confirmed and auto_confirmed.get("error_detail"):
|
||||
redirect_url += "&paperless_error=" + quote_plus(str(auto_confirmed["error_detail"]))
|
||||
return RedirectResponse(url=redirect_url, status_code=303)
|
||||
return response_payload
|
||||
|
||||
|
||||
@@ -1,12 +1,19 @@
|
||||
{% extends 'base.html' %}
|
||||
{% block content %}
|
||||
{% if request.query_params.get('submitted') %}
|
||||
<section class="card form-card notice-card success-card evidence-board">
|
||||
<div class="case-legend"><span class="section-label">Submission packet</span><span class="pill">Filed</span></div>
|
||||
<h2>Submission Created</h2>
|
||||
{% set paperless_status = request.query_params.get('paperless_status', 'uploaded-local') %}
|
||||
<section class="card form-card notice-card {{ 'success-card' if paperless_status == 'paperless-queued' else '' }} evidence-board">
|
||||
<div class="case-legend"><span class="section-label">Submission packet</span><span class="pill">{{ 'Filed' if paperless_status == 'paperless-queued' else 'Saved locally' }}</span></div>
|
||||
<h2>{{ 'Submission Created' if paperless_status == 'paperless-queued' else 'Submission Saved, Archive Pending' }}</h2>
|
||||
<p>Submission {{ request.query_params.get('submitted') }} was created successfully.</p>
|
||||
<p class="muted">Artifact: {{ request.query_params.get('artifact_kind', 'document') }} · Files: {{ request.query_params.get('file_count', '1') }}</p>
|
||||
<p class="muted">Paperless status: {{ request.query_params.get('paperless_status', 'uploaded-local') }}</p>
|
||||
<p class="muted">Paperless status: {{ paperless_status }}</p>
|
||||
{% if paperless_status != 'paperless-queued' %}
|
||||
<p class="muted">Schoolhouse kept the submission locally, but the archive handoff failed this time.</p>
|
||||
{% if request.query_params.get('paperless_error') %}
|
||||
<p class="muted">Error: {{ request.query_params.get('paperless_error') }}</p>
|
||||
{% endif %}
|
||||
{% endif %}
|
||||
</section>
|
||||
{% endif %}
|
||||
<section class="card form-card evidence-board">
|
||||
|
||||
@@ -27,6 +27,12 @@ Basic verification after updates:
|
||||
- optional functional scrape smoke test:
|
||||
- POST `{"url":"https://example.com"}` to `http://127.0.0.1:3302/v1/scrape`
|
||||
|
||||
Firecrawl stability note on PD:
|
||||
- The upstream `harness.js --start-docker` launcher fans out into API + queue worker + extract worker + NUQ workers.
|
||||
- On PD, the default fan-out has caused first-start flaps (`Port 3002 did not become available within 60000ms`) and steady `WORKER STALLED` / `Can't accept connection due to RAM/CPU load` messages.
|
||||
- Keep the PD compose defaults conservative unless the host is re-sized: `NUM_WORKERS_PER_QUEUE=1`, `CRAWL_CONCURRENT_REQUESTS=1`, `MAX_CONCURRENT_JOBS=1`, `BROWSER_POOL_SIZE=1`, `NUQ_WORKER_COUNT=1`, and `HARNESS_STARTUP_TIMEOUT_MS=180000`.
|
||||
- After changing those values, force-recreate `firecrawl-api` and confirm `curl -sS http://127.0.0.1:3302/` answers on the first start without an immediate harness timeout.
|
||||
|
||||
Rollback-first notes:
|
||||
- Back up `docker-compose.yaml` before live edits under `/home/truenas_admin/doris-backups/`
|
||||
- Back up `settings.yml` before SearXNG config edits under `/home/truenas_admin/doris-backups/`
|
||||
|
||||
@@ -126,10 +126,11 @@ services:
|
||||
- POSTGRES_HOST=firecrawl-postgres
|
||||
- POSTGRES_PORT=5432
|
||||
- USE_DB_AUTHENTICATION=${USE_DB_AUTHENTICATION:-false}
|
||||
- NUM_WORKERS_PER_QUEUE=${NUM_WORKERS_PER_QUEUE:-4}
|
||||
- CRAWL_CONCURRENT_REQUESTS=${CRAWL_CONCURRENT_REQUESTS:-4}
|
||||
- MAX_CONCURRENT_JOBS=${MAX_CONCURRENT_JOBS:-2}
|
||||
- BROWSER_POOL_SIZE=${BROWSER_POOL_SIZE:-2}
|
||||
- NUM_WORKERS_PER_QUEUE=${NUM_WORKERS_PER_QUEUE:-1}
|
||||
- CRAWL_CONCURRENT_REQUESTS=${CRAWL_CONCURRENT_REQUESTS:-1}
|
||||
- MAX_CONCURRENT_JOBS=${MAX_CONCURRENT_JOBS:-1}
|
||||
- BROWSER_POOL_SIZE=${BROWSER_POOL_SIZE:-1}
|
||||
- NUQ_WORKER_COUNT=${NUQ_WORKER_COUNT:-1}
|
||||
- BULL_AUTH_KEY=${BULL_AUTH_KEY}
|
||||
- TEST_API_KEY=${TEST_API_KEY}
|
||||
- LOGGING_LEVEL=${LOGGING_LEVEL:-info}
|
||||
@@ -138,7 +139,7 @@ services:
|
||||
- SEARXNG_CATEGORIES=${SEARXNG_CATEGORIES:-}
|
||||
- ALLOW_LOCAL_WEBHOOKS=${ALLOW_LOCAL_WEBHOOKS:-false}
|
||||
- BLOCK_MEDIA=${BLOCK_MEDIA:-true}
|
||||
- HARNESS_STARTUP_TIMEOUT_MS=60000
|
||||
- HARNESS_STARTUP_TIMEOUT_MS=${HARNESS_STARTUP_TIMEOUT_MS:-180000}
|
||||
- NUQ_RABBITMQ_URL=amqp://firecrawl-rabbitmq:5672
|
||||
command: node dist/src/harness.js --start-docker
|
||||
|
||||
|
||||
Reference in New Issue
Block a user