Skip to content

Stop supporting user-defined error decoders #80

@whickman

Description

@whickman

What happened?

Custom error decoders are able to arbitrarily modify request and response bodies. This produces a huge API surface and a huge potential spread of client behavior. We should remove this ability and become opinionated about error handling.

This blocks moving retry behavior into middleware before error decoding, as retry behavior depends on whether or not an error was returned.

Ideally, we would also remove the WithDisableRestErrors functionality, although that will require more work.

What did you want to happen?

Users should not be able to define custom decoders.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions