nextcloud / nextcloud/android

Passing Full OCFile to EditorWebView May Exceed Parcelable Limit and Cause App Crash

Open
#14,852 0 comments 0 reactions 1 assignee Claimed by @alperozturk96 View on GitHub
bug
Dominant language
Kotlin
Stars
5.6k
Forks
2k
Avg merge
2d 18h
Merged PRs (30d)
92

Description

### ⚠️ Before posting ⚠️

- [x] This is a **bug**, not a question or an enhancement.
- [x] I've [searched for similar issues](https://github.com/nextcloud/android/issues) and didn't find a duplicate.
- [x] I've written a clear and descriptive title for this issue, not just "Bug" or "Crash".
- [x] I agree to follow Nextcloud's [Code of Conduct](https://nextcloud.com/contribute/code-of-conduct/).

### Steps to reproduce

Attempt to open a file with extensive properties or content. e.g. big XLSX file

### Expected behaviour

The file should open in the editor without crashing. File ID should be used as argument instead of file itself.

### Actual behaviour

If the OCFile exceeds the 1MB limit for Android Parcelable objects, the app crashes when attempting to pass it to EditorWebView or its references.

### Android version

15

### Device brand and model

Android Emulator

### Stock or custom OS?

Stock

### Nextcloud android app version

3.31.0

### Nextcloud server version

31.0.0

### Using a reverse proxy?

I don't know

### Android logs

_No response_

### Server error logs

```bash

```

### Additional information

ChooseRichDocumentsTemplateDialogFragment#L399
EditorWebView#L244
ChooseTemplateDialogFragment#L349

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.