Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[New article]: Use EndpointProperty.HostAndPort in samples #2646

Open
afscrome opened this issue Feb 20, 2025 · 1 comment
Open

[New article]: Use EndpointProperty.HostAndPort in samples #2646

afscrome opened this issue Feb 20, 2025 · 1 comment
Labels
doc-idea Indicates issues that are suggestions for new topics [org][type][category] ⌚ Not Triaged Not triaged

Comments

@afscrome
Copy link
Contributor

Proposed topic or title

NA - updates to existing articles

Location in table of contents.

N/A

Reason for the article

dotnet/aspire#7629 introduces an EndpointProperty.HostAndPort to 9.2 that can simplify scenarios where you previously needed to concatenate Host and Port seperately - {foo.Property(EndpointProperty.Host)}:{foo.Property(EndpointProperty.Port)}

The docs currently have a few places that could use this simplification

$"smtp://{SmtpEndpoint.Property(EndpointProperty.Host)}:{SmtpEndpoint.Property(EndpointProperty.Port)}"

$"smtp://{SmtpEndpoint.Property(EndpointProperty.Host)}:{SmtpEndpoint.Property(EndpointProperty.Port)}"

$"smtp://{SmtpEndpoint.Property(EndpointProperty.Host)}:{SmtpEndpoint.Property(EndpointProperty.Port)}"

$"Endpoint=smtp://{SmtpEndpoint.Property(EndpointProperty.Host)}:{SmtpEndpoint.Property(EndpointProperty.Port)};Username={UserNameReference};Password={PasswordParameter}"

Article abstract

N/A

Relevant searches

Searching through the repo for .Property(EndpointProperty.Host)}:{ is how I found the above in case more of these show up in between creating this issue and the docs being updated for 9.2.

@afscrome afscrome added the doc-idea Indicates issues that are suggestions for new topics [org][type][category] label Feb 20, 2025
@dotnetrepoman dotnetrepoman bot added the ⌚ Not Triaged Not triaged label Feb 20, 2025
@davidfowl
Copy link
Member

davidfowl commented Feb 20, 2025

We need to write the document about endpoint references in the first place 😇 (also this is 9.2 content)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-idea Indicates issues that are suggestions for new topics [org][type][category] ⌚ Not Triaged Not triaged
Projects
None yet
Development

No branches or pull requests

2 participants