dapr / dapr/proposals

"http://" to default to port 80 in gRPC endpoint

Open
#56 3 comments 3 reactions 0 assignees View on GitHub
Dominant language
No language data
Stars
17
Forks
39
PR merge metrics
No merged PRs in 30d

Description

Our custom logic for gRPC endpoint parsing default "http" to 443, which is not compatible to how users would expect the URL spec to behave. I understand that the gRPC endpoint and URL spec are not the same thing but when an user writes the environment variable for gRPC with HTTP URL terminology, we should respect the default ports expected as per the standard URL RFC (needs link to RFC). Defaulting to port 443 does not bring any extra security, since the port itself does not mean TLS, it is just a convention - a service can perfectly run without TLS on port 443. IMO, going against the RFC is not user-friendly for default port convention.

This proposal is to change the expected port to 80 when the user specifies "http://" as the prefix for the gRPC endpoint: https://github.com/dapr/proposals/blob/main/20230627-S-sidecar-endpoint-tls.md#test-matrix

I would like to collect quantitative data about how the community feels one way or another.

👍 For this proposal
👎 To keep it as-is

Contributor guide

No contributing guide indexed for this repository

Research direction

Read the linked sidecar endpoint TLS test matrix and compare its HTTP-port expectations with the URL RFC behavior described here. Review the discussion and reaction tally; done means documenting a decision on whether http:// should imply port 80, rather than assuming the change is ready to implement.

Written by the indexing model from the issue text.

Assessment

Tech stack
grpc
Domain
backend-api-design, networking
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.