Fix http json example

[ramgdev]

Fixes #1763
Design discussion issue (if applicable) #

Changes

Use string value for event attribute

Merge requirement checklist

• CONTRIBUTING guidelines followed
• Unit tests added/updated (if applicable)
• Appropriate CHANGELOG.md files updated for non-trivial, user-facing changes
• Changes in public API reviewed (if applicable)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.0%. Comparing base (0303a38) to head (fb950df).
Report is 4 commits behind head on main.

❗ Current head fb950df differs from pull request most recent head 028ad0e

Please upload reports for the commit 028ad0e to get more accurate results.

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #1835   +/-   ##
=====================================
  Coverage   75.0%   75.0%           
=====================================
  Files        122     122           
  Lines      20279   20279           
=====================================
+ Hits       15211   15212    +1     
+ Misses      5068    5067    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ramgdev]

@lalitb @TommyCpp please take a look at the changes when you get a chance. I had to disable the call to serialize int values to string so it works correctly when deserializing from JSON. I think JSON deserializer expects StringValue for all string attribute values.

My editor picked up some formatting changes in addition.

[lalitb]

@ramgdev Please go through the fix mentioned in the comment here. Do you think it resolves this issue also ?

[ramgdev]

@ramgdev Please go through the fix mentioned in the comment here. Do you think it resolves this issue also ?

@lalitb thanks. It looks like it will take care of this issue as well. I will wait to test it after that PR merges and remove the edits from proto.rs in this PR.

[lalitb]

@ramgdev Please go through the fix mentioned in the comment here. Do you think it resolves this issue also ?

@lalitb thanks. It looks like it will take care of this issue as well. I will wait to test it after that PR merges and remove the edits from proto.rs in this PR.

@ramgdev fyi - #1882 is merged.

[lalitb]

LGTM with nit change. Thanks for the PR.

[ramgdev]

closing to open a separate PR