Lorawan-stack: Rendu enum incohérent dans les réponses JSON

Sommaire

Références : https://github.com/TheThingsNetwork/lorawan-stack/issues/2047

Les valeurs renvoyées par NS pour les énumérations dans le champ mac_settings sont incohérentes. Certains pour certains NS renvoient des valeurs brutes et pour d'autres une représentation sous forme de chaîne est renvoyée.

Étapes à suivre pour reproduire

1. Définir mac_settings.rx1_delay.value

// request
{
   "end_device":{
      "mac_settings":{
         "rx1_delay":{
            "value":5 // or 'RX_DELAY_5'
         }
      }
   },
   "field_mask":{
      "paths":[
         "mac_settings.rx1_delay.value",
      ]
   }
}

// response
{
   ...
   "mac_settings":{
      "rx1_delay":{
         "value":5
      },
   }
}

// Note, regardless of what you send as the value for mac_settings.rx1_delay.value you always get the raw enum value.

1. Ensemble mac_settings.ping_slot_periodicity.value

// request

{
   "end_device":{
      "mac_settings":{
         "ping_slot_periodicity":{
            "value": 1 // or 'PING_EVERY_2S'
         }
      }
   },
   "field_mask":{
      "paths":[
         "mac_settings.ping_slot_periodicity.value"
      ]
   }
}

// response

{
   "mac_settings":{
      "ping_slot_periodicity":{
         "value":"PING_EVERY_2S"
      }
   }
}

// Again, regardless of what you send you always get the string representation of the value.

Que voyez-vous maintenant ?

Valeurs incohérentes pour les champs d'énumération mac_settings

Que veux-tu voir à la place ?

Toutes les valeurs doivent être renvoyées dans le même format (brut/chaîne)

Environnement

v3.7.0-rc2

Comment proposez-vous de mettre cela en œuvre ?

Je suppose que nous devons d'abord nous mettre d'accord sur le format. Ensuite, il s'agit d'ajuster https://github.com/TheThingsNetwork/lorawan-stack/blob/master/api/lorawan.proto et https://github.com/TheThingsNetwork/lorawan-stack/blob/master/pkg /ttnpb/lorawan.go

Pouvez-vous le faire vous-même et soumettre une demande d'extraction ?

Oui, mais besoin de la contribution de @rvolosatovs @johanstokking @htdvisser