Skip to content

Excel export can leak local paths via attachment metadata #23

Description

@JeromeMaag

The Excel export currently does not strip external_path from attachment metadata.

Current flow:

  • threema/media/export.py sets external_path
  • threema/normalize.py stores it in attachment.metadata
  • render/excel_builder.py sanitizes some path fields, but not external_path

Reproduced

A minimal workbook built with:

attachment.metadata = {"external_path": "C:/Users/secret/_EXTERNAL_DATA/secret.png"}

still contained that exact path inside the generated workbook XML.

Problem

This can leak local absolute paths and user/environment information into the Excel export.

Expected

external_path should be removed from Excel metadata output just like other absolute local path fields.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions