getappmap / appmap-agent-js

This project is deprecated. Please use https://github.com/getappmap/appmap-node/ to record Node.js applications.
Other
28 stars 8 forks source link

Fix trace digest #161

Closed lachrist closed 1 year ago

lachrist commented 1 year ago

fixes https://github.com/getappmap/appmap-agent-js/issues/159

It was assumed that the number of recorded arguments is always the same as the number of parameters. This is actually not the case when, in the presence of source mapping, the original function does not have the same number of parameters of the generated function. For instance, this can be the case when the agent could not find a function at the indicated position and pick the next best guest. To solve this issue we simply truncate the zip to the smallest array.

appland-release commented 1 year ago

:tada: This PR is included in version 12.1.2 :tada:

The release is available on:

Your semantic-release bot :package::rocket: