George Bashi 9c93f5d4cb fix: prevent null cache poisoning from permanently hiding channels ...

When the Slack API returned no channels (due to upstream failures),
json.MarshalIndent(nil) produced "null" which was written to the
cache file. On subsequent startups, json.Unmarshal("null") into a
slice succeeds with no error, producing an empty slice that was
treated as valid cached data — permanently serving empty results
until the cache TTL expired.

Three defense-in-depth guards:
- Don't write cache file when channel list is empty
- Treat empty/null unmarshaled channels cache as invalid (refetch)
- Same empty-cache guard for users cache (consistency)

2026-03-03 11:31:39 -08:00

3.0 KiB

Raw Permalink Blame History

View Raw