Usage data payload reply_by_email_enabled possible issue

Noticed this in raw usage data payload

“reply_by_email_enabled”: “incoming+%{key}@incoming.gitlab.com”,

this should be a boolean or nil, or not? this is also in the example of usage ping in docs

https://docs.gitlab.com/ee/development/telemetry/usage_ping.html#example-usage-ping-payload

Kathleen Tam: do we know if this is how we default it? or is there a way we can confirm a user is passing bad data? cc: @derek / @jeromezng on our convo on bad usage data from users. FYI @Rob Parker


Jerome:  6 hours ago
@Alina looking at the method, it looks incorrect. I think we should separate this into two fields instead
"reply_by_email_enabled" : true (boolean)
"reply_by_email" : "incoming+%{key}@incoming.gitlab.com" (string) 

cc @jeromezng @sid_reddy @kathleentam