dheerenmohta opened a new pull request, #71885: URL: https://github.com/apache/airflow/pull/71885
Services like Matrix rate-limit repeated logins, and Apprise avoids that by caching session tokens to disk via an `AppriseAsset` with a `storage_path`. `AppriseHook` never built one, so every `notify()` call re-authenticated from scratch. This lets callers opt in with `storage_path`/`storage_mode` constructor kwargs, mirroring how `apprise_conn_id` is already configured on this hook; default behavior is unchanged when `storage_path` is left unset. apprise only gained `storage_path`/`storage_mode`/`PersistentStoreMode` in 1.9.0 (verified against the published wheels for 1.8.0, 1.8.1 and 1.9.0), so the provider's minimum apprise pin moves from `>=1.8.0` to `>=1.9.0` to match. Two earlier PRs against this issue (#51105, #63741) stalled on responsiveness rather than a design objection; #63741's constructor-kwarg approach was sound but silently depended on this missing version bump, which is the most likely reason its mypy check failed in CI. closes: #51102 related: #51105, #63741 --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Sonnet 5) Generated-by: Claude Code (Sonnet 5) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
