Skip to content

Remove mobile-config from mobile-verifier - #1228

Merged
michaeldjeffrey merged 4 commits into
mainfrom
mj/remove-mobile-config-from-verifier
Jul 30, 2026
Merged

Remove mobile-config from mobile-verifier#1228
michaeldjeffrey merged 4 commits into
mainfrom
mj/remove-mobile-config-from-verifier

Conversation

@michaeldjeffrey

Copy link
Copy Markdown
Contributor
  • Gateway resolution now done through trino, similar to mobile-packet-verifier.
  • Authorization now done with hardcoded list of valid keys in settings, similar to mobile-packet-verifier.
  • EpochRewardInfo + EpochInfo + DeviceType pulled from mobile-config, values come from trino.
  • SETTINGS:
    • remove config_client
    • add gateway_refresh_interval
    • add banning_authorized_keys
    • add mobile_carrier_authorized_keys

@michaeldjeffrey
michaeldjeffrey requested a review from bbalser July 28, 2026 19:21
Base automatically changed from mj/calculate-hnt-price to main July 28, 2026 19:30
- Gateway resolution now done through trino, similar to mobile-packet-verifier.
- Authorization now done with hardcoded list of valid keys in settings, similar to mobile-packet-verifier.
- EpochRewardInfo + EpochInfo + DeviceType pulled from mobile-config, values come from trino.
- SETTINGS:
  - remove `config_client`
  - add `gateway_refresh_interval`
  - add `banning_authorized_keys`
  - add `mobile_carrier_authorized_keys`
@michaeldjeffrey
michaeldjeffrey force-pushed the mj/remove-mobile-config-from-verifier branch from f8ffa8c to e36283f Compare July 28, 2026 19:38
Verified there are no rows with NULL device_type.
Add a stub test to manually verify against a prod trino
@michaeldjeffrey
michaeldjeffrey force-pushed the mj/remove-mobile-config-from-verifier branch from c209752 to 5e92507 Compare July 28, 2026 22:44
@michaeldjeffrey
michaeldjeffrey merged commit c978779 into main Jul 30, 2026
28 checks passed
@michaeldjeffrey
michaeldjeffrey deleted the mj/remove-mobile-config-from-verifier branch July 30, 2026 19:15
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.

2 participants