Skip to content

Naming inconsistencies for HTTP filters #119

Open
@NiteshKant

Description

@NiteshKant

HTTP APIs follow the convention:

  • For aggregated APIs (non-streaming): No qualification is present in the names. eg: HttpRequest, HttpResponse.
  • For async streaming APIs: We prefix with Streaming. eg: StreamingHttpRequest, StreamingHttpResponse
  • For blocking streaming APIs: We prefix with BlockingStreaming. eg: BlockingStreamingHttpRequest, BlockingStreamingHttpResponse

However, we only provide ways to add async streaming filters, so we drop the qualification in the name. eg: TracingHttpServiceFilter extends StreamingHttpService but it does not have a name qualification according to the above convention. The rational is to reduce naming verbosity at the cost of not following the conventions.

Not following conventions can cause confusion for users and we should revisit to see if we can reduce this confusion.

Metadata

Metadata

Assignees

No one assigned

    Labels

    APIPR with API changes (New or Deprecated)

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions