Closed jstarpl closed 3 weeks ago
Attention: Patch coverage is 14.28571%
with 30 lines
in your changes missing coverage. Please review.
Project coverage is 57.99%. Comparing base (
e84c76a
) to head (c5d8c7e
).
Files | Patch % | Lines |
---|---|---|
meteor/server/api/rest/koa.ts | 15.15% | 28 Missing :warning: |
meteor/server/api/rest/v1/index.ts | 0.00% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
About the Contributor
This is a bugfix posted on behalf of NRK
Type of Contribution
This is a:
Bug fix
Current Behavior
The UserActions Log UserEvent string is not very convenient. Also, the remoteAddress information is not extracted from
Forwarded
HTTP headers.New Behavior
The logged remote address should match the "public" IP address of the callee. The UserActions Log userEvent should be more sensible (REST API, method, endpoint, host)
Testing
Affected areas
Time Frame
Other Information
Status