Skip to content

Commit 34b95e4

Browse files
committed
Fix security warnings
1 parent d5ff5e5 commit 34b95e4

File tree

2 files changed

+13
-5
lines changed

2 files changed

+13
-5
lines changed

Dockerfile

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,14 @@ COPY --from=ghcr.io/astral-sh/uv:latest /uv /uvx /bin/
44

55
WORKDIR /app
66

7-
COPY . .
7+
RUN groupadd --system --gid 1001 appgroup && \
8+
useradd --system --uid 1001 --gid appgroup appuser
9+
10+
RUN chown -R appuser:appgroup /app
11+
12+
COPY --chown=appuser:appgroup . .
13+
14+
USER appuser
815

916
RUN uv sync --frozen --no-cache
1017

pdf.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -35,12 +35,13 @@ async def page_requests(headers, url, file):
3535
pdf_writer = PdfWriter()
3636
pdf_writer.add_page(pdf.pages[page])
3737

38-
out_file_name = os.path.join(temp_dir, f"page-{page:05d}.pdf")
38+
out_file_name = os.path.abspath(os.path.join(temp_dir, f"page-{page:05d}.pdf"))
3939

40-
with open(out_file_name, 'wb') as out:
41-
pdf_writer.write(out)
40+
if out_file_name.startswith(temp_dir):
41+
with open(out_file_name, 'wb') as out:
42+
pdf_writer.write(out)
4243

43-
pdf_pages.append(out_file_name)
44+
pdf_pages.append(out_file_name)
4445

4546
pages = len(pdf_pages)
4647

0 commit comments

Comments
 (0)