search

exceptionless / Exceptionless.Net

Exceptionless clients for the .NET platform
https://exceptionless.com
Other
557 stars 142 forks source link

Fixes #127: Added ability to record http headers #300

Closed niemyjski closed 1 year ago

niemyjski commented 1 year ago

Before merging this we probably need to make these changes server side as well.

I also want to add MAX_DATA_ITEM_LENGTH to the other two implementations.

ejsmith commented 1 year ago

This looks good. Not sure how the string[] array will get handled by Exceptionless though. Should we just have it be a comma delimited string of header values?

niemyjski commented 1 year ago

We need to update the backend first, but I'd rather do an array here than a comma delimited list as it would be easier to render out and then you have no issues escaping existing values that might have a comma.