Fix charge_when_no_offer (#717)

* Fix charge_when_no_offer * Fix status badge
helium · May 2, 2022 · 3d50ac3 · 3d50ac3
1 parent 711e7a6
commit 3d50ac3
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -1,4 +1,4 @@
-# Helium Router [![Tests](https://github.com/helium/router/actions/workflows/main.yml/badge.svg)](https://github.com/helium/router/actions/workflows/main.yml)
+# Helium Router [![Tests](https://github.com/helium/router/actions/workflows/tests.yml/badge.svg)](https://github.com/helium/router/actions/workflows/tests.yml)
 
 Helium's LoRa Network Server (LNS) backend.
 

diff --git a/src/device/router_device_worker.erl b/src/device/router_device_worker.erl
@@ -1569,7 +1569,7 @@ validate_frame_(Packet, PubKeyBin, HotspotRegion, Device0, OfferCache, Blockchai
 maybe_charge(Device, PayloadSize, Blockchain, PubKeyBin, PHash, OfferCache) ->
     case maps:get({PubKeyBin, PHash}, OfferCache, undefined) of
         undefined ->
-            case application:get_env(router, charge_when_no_offer, true) of
+            case charge_when_no_offer() of
                 false ->
                     Metadata = router_device:metadata(Device),
                     {Balance, Nonce} =
@@ -1587,6 +1587,14 @@ maybe_charge(Device, PayloadSize, Blockchain, PubKeyBin, PHash, OfferCache) ->
             router_console_dc_tracker:charge(Device, PayloadSize, Blockchain)
     end.
 
+-spec charge_when_no_offer() -> boolean().
+charge_when_no_offer() ->
+    case application:get_env(router, charge_when_no_offer, true) of
+        "true" -> true;
+        true -> true;
+        _ -> false
+    end.
+
 %%%-------------------------------------------------------------------
 %% @doc
 %% Check device's message queue to potentially wait or send reply