microsoft / fhir-server

A service that implements the FHIR standard
MIT License
1.15k stars 490 forks source link

Update $convert-data error log #3859

Closed pallar-ms closed 1 month ago

pallar-ms commented 1 month ago

Description

Update $convert-data error log. Currently, the entire exception is logged in some cases (except post processing exceptions) which might not be ideal in case the message has text from the input data being processed. Updating the log to only mention exception type, FHIRConverterErrorCode and stack trace.

Related issues

Addresses AB#120207.

Testing

Describe how this change was tested.

FHIR Team Checklist

Semver Change (docs)

Patch|Skip|Feature|Breaking (reason)

pallar-ms commented 1 month ago

@LTA-Thinking , thanks for taking a look. Question - the PR build & deploy seems to be failing at an unrelated change to this PR, and I am also seeing other PRs blocked by the same error. Do you know if anyone is taking a look into that? What is the best place to bring this up and track status?