bugfix: APPRISE_ATTACH_SIZE correctly manages attachment size - Githubissues

caronc / apprise-api

A lightweight REST framework that wraps the Apprise Notification Library

https://hub.docker.com/r/caronc/apprise

MIT License

638 stars 56 forks source link

bugfix: APPRISE_ATTACH_SIZE correctly manages attachment size #177

Closed caronc closed 6 months ago

caronc commented 6 months ago

Description:

Related issue (if applicable): refs #176

The APPRISE_ATTACH_SIZE global correctly handles attachment sizes again (defined in MB) and defaults to 200.

Previous to this there was another variable (undocumented) fixing this value to 8MB.

This PR also makes server side logging a bit more explanatory as to why the attachment was not processed (should this be the case)

Checklist

[x] The code change is tested and works locally.
[x] There is no commented out code in this PR.
[x] No lint errors (use flake8)
[x] Tests added