Skip to content

Update DEFAULT_HTTP_CLIENT_ERROR_STATUSES to include 5xx status codes #10245

@brentm5

Description

@brentm5

Library Name

No response

Library Version(s)

No response

Describe the feature you'd like

Update the default HTTP client error status codes to include 5xx errors.

DEFAULT_HTTP_CLIENT_ERROR_STATUSES = new BitSet();
DEFAULT_HTTP_CLIENT_ERROR_STATUSES.set(400, 500);

Is your feature request related to a problem?

No response

Describe alternatives you've considered

No response

Additional context

With the addition of inferred services, trace metrics from http clients will show up as part of the calling service. These metrics are used to show latency metrics to downstream services. Right now the errors that are shown only apply to 4xx errors. This is misleading as 5xx errors do not show in the UI as an error for downstream calls to other services.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions