Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Attachment shows "link expired" for external attachments #529

Closed
binwiederhier opened this issue Dec 5, 2022 · 3 comments
Closed

Attachment shows "link expired" for external attachments #529

binwiederhier opened this issue Dec 5, 2022 · 3 comments
Labels
android-app ntfy Android app 🪲 bug Something isn't working

Comments

@binwiederhier
Copy link
Owner

In the case below, the attachment can only be downloaded from the notification action button, but not from the ntfy detail view.

curl -d "Here's a notification with an .apk attachment" -H "Attach: https://f-droid.org/repo/com.calcitem.sanmill_3178.apk" -H p:4 ntfy.sh/demogoogle

Here's a video+screenshot showing the issue. The first attempt in the video works, the following attempts do not: https://photos.app.goo.gl/mKAEyh7VYyWBmtaW6

image

@binwiederhier binwiederhier added 🪲 bug Something isn't working android-app ntfy Android app labels Dec 5, 2022
@binwiederhier
Copy link
Owner Author

I think this is a Firebase-only bug. Top one received via instant delivery, bottom via Firebase:

image

@binwiederhier
Copy link
Owner Author

It's because attachment_expires=0:

2022-12-04 23:11:45.920 2619-5880/io.heckel.ntfy D/NtfyFirebase: Received message: from=/topics/demogoogle, fcmprio=1, fcmprio_orig=1, data={priority=4, attachment_name=com.calcitem.sanmill_3178.apk, attachment_size=0, attachment_type=, attachment_expires=0, id=lpClKDAT6eJz, icon=, tags=, time=1670213505, click=, event=message, title=, topic=demogoogle, attachment_url=https://f-droid.org/repo/com.calcitem.sanmill_3178.apk, message=Here's a notification with an .apk attachment, encoding=}

@binwiederhier
Copy link
Owner Author

Fixed in binwiederhier/ntfy-android@b91778a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android-app ntfy Android app 🪲 bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant