Skip to content

fix: detect octet-stream ICS attachments - #13732

Merged
kesselb merged 1 commit into
nextcloud:mainfrom
tayfuryldz:fix/13688-octet-stream-ics
Sep 28, 2026
Merged

kesselb merged 1 commit into
nextcloud:mainfrom
tayfuryldz:fix/13688-octet-stream-ics

Conversation

@tayfuryldz

Copy link
Copy Markdown
Contributor

Fixes #13688

Summary

  • treat application/octet-stream attachments as calendar events only when their filename ends in .ics
  • keep existing text/calendar and application/ics behavior unchanged
  • add focused coverage for uppercase .ICS and negative MIME/extension cases

Validation

  • git diff --check
  • PHP is not installed in this local environment, so PHPUnit and php -l could not be run locally; CI is left to perform PHP validation

@welcome

welcome Bot commented Sep 26, 2026

Copy link
Copy Markdown

Thanks for opening your first pull request in this repository! 鉁岋笍

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

馃煝 Approval recommended

No unresolved review issues were identified.

Review effort: Lite
Findings: None

What changed in this PR

Updates calendar attachment detection for .ics files labeled application/octet-stream, preserving existing MIME behavior.

Changes:

  • Added case-insensitive .ics filename detection.
  • Added positive and negative unit-test coverage.
File Description
tests/鈥婾nit/鈥婥ontroller/鈥婱essagesControllerTest.php Covers MIME and extension combinations.
lib/鈥婥ontroller/鈥婱essagesController.php Detects octet-stream ICS attachments.

馃挕 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@kesselb

kesselb commented Sep 26, 2026

Copy link
Copy Markdown
Contributor

Thanks a lot for your pr 馃憤

Looks good 馃檹

Please sign off your commits: https://github.com/nextcloud/mail/pull/13732/checks?check_run_id=108473040545

If AI was used to create this PR, please add a Assisted-By trailer to the commit message.

More details at: https://github.com/nextcloud/mail/blob/main/AGENTS.md#commit-message-format

Assisted-by: ChatGPT:gpt-5.6-sol
Signed-off-by: tayfuryldzz <yildiztayfur668@gmail.com>
@tayfuryldz
tayfuryldz force-pushed the fix/13688-octet-stream-ics branch from 48b7730 to f4374d8 Compare September 28, 2026 18:01
@tayfuryldz

Copy link
Copy Markdown
Contributor Author

Updated the commit trailer to the AGENTS.md format and kept the DCO Signed-off-by trailer. The branch now points to f4374d8.

@kesselb

kesselb commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

/backport to stable5.12 please

@kesselb
kesselb enabled auto-merge September 28, 2026 18:33
@kesselb
kesselb merged commit 45d3c5f into nextcloud:main Sep 28, 2026
47 of 48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

application/octet-stream .ics attachment not detected as event

3 participants