Skip to content

Conversation

@lemnik
Copy link
Contributor

@lemnik lemnik commented Oct 24, 2025

Goal

Introduce the Hash Discard rule to the RemoteConfig capabilities.

Changes

  • added the Hash discard rule to the RemoteConfig class
  • Hash objects do not parse their DiscardRule content until they are first used
  • Changed the All and AllHandled discard rules to be objects instead of classes

Testing

Added new unit tests for the Hash parsing and serialization. Also tested manually.

@bugsnagbot
Copy link
Collaborator

bugsnagbot commented Oct 24, 2025

Android notifier sizes

Format Size impact of Bugsnag (kB) Size impact of Bugsnag when Minified (kB)
APK 1858.94 1603.07
arm64_v8a 667.91 409.86
armeabi_v7a 610.57 352.52
x86 741.62 483.57
x86_64 712.95 454.9

Generated by 🚫 Danger

@lemnik lemnik force-pushed the PLAT-15072/hash-discard-rule branch from 18a6537 to f8120cc Compare November 5, 2025 09:18
@lemnik lemnik force-pushed the PLAT-15072/hash-discard-rule branch from f8120cc to 116fd9b Compare November 5, 2025 09:22
@lemnik lemnik marked this pull request as ready for review November 5, 2025 09:24
@lemnik lemnik requested a review from YYChen01988 as a code owner November 5, 2025 09:24
@lemnik lemnik force-pushed the PLAT-15072/hash-discard-rule branch from aefc51e to 664580b Compare November 5, 2025 10:06
@lemnik lemnik marked this pull request as draft November 5, 2025 13:57
…ig with an empty ruleset until the next check time
@lemnik lemnik force-pushed the PLAT-15072/hash-discard-rule branch from 6259c7e to f5c602c Compare November 5, 2025 15:42
…erver to either define whether the etags are expected quoted or not
@lemnik lemnik force-pushed the PLAT-15072/hash-discard-rule branch from f49b7d5 to 7c7f4ad Compare November 7, 2025 08:20
…down to 0, remoteconfig expires exactly when the Cache-Control header says it does
@lemnik lemnik force-pushed the PLAT-15072/hash-discard-rule branch from 7e12c23 to b6dd888 Compare November 7, 2025 10:19
@lemnik lemnik force-pushed the PLAT-15072/hash-discard-rule branch from b6dd888 to 339a8f4 Compare November 7, 2025 10:44
@lemnik lemnik marked this pull request as ready for review November 7, 2025 12:38
DariaKunoichi
DariaKunoichi previously approved these changes Nov 7, 2025
@lemnik lemnik requested a review from YYChen01988 November 10, 2025 08:17
tomlongridge
tomlongridge previously approved these changes Nov 12, 2025
@lemnik lemnik dismissed stale reviews from tomlongridge and DariaKunoichi via 6b5958a November 18, 2025 10:39
@lemnik lemnik merged commit 3a03940 into integration/remote-config Nov 18, 2025
37 checks passed
@lemnik lemnik deleted the PLAT-15072/hash-discard-rule branch November 18, 2025 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants