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

Self-describing minimal APIs with IEndpointMetadataProvider and IEndpointParameterMetadataProvider #76

Closed
Rick-Anderson opened this issue Sep 28, 2022 · 13 comments

Comments

@Rick-Anderson
Copy link
Contributor

Rick-Anderson commented Sep 28, 2022

Create sample code. See https://devblogs.microsoft.com/dotnet/asp-net-core-updates-in-dotnet-7-preview-4/

Possibly same code can do dotnet/sdk#77

@sammychinedu2ky
Copy link
Contributor

Alright, @Rick-Anderson.
Please where should it be located

@sammychinedu2ky
Copy link
Contributor

sammychinedu2ky commented Sep 29, 2022

@Rick-Anderson I noticed a little error.
Port 5060 and 5061 aren't allowed by new browser versions due to the natslip attack . Port 5060 was randomly created for me in the LaunchSettings.json file which made me have issues accessing my API from the browser. I had to manually change the applicationUrl port

@Rick-Anderson
Copy link
Contributor Author

@Tratcher where should this be reported?

@sammychinedu2ky
Copy link
Contributor

@Tratcher where should this be reported?

Please @Tratcher, if you can show me where the port generation is done in the dotnet SDK, I would also appreciate it

@Tratcher
Copy link
Member

File an issue in https://github.com/dotnet/sdk/ and we'll address it from there.

@sammychinedu2ky
Copy link
Contributor

File an issue in https://github.com/dotnet/sdk/ and we'll address it from there.

alright thanks

@sammychinedu2ky
Copy link
Contributor

if you can show me where the port generation is done in the dotnet SDK, I would also appreciate it

thanks done

@Rick-Anderson
Copy link
Contributor Author

dotnet/templating#5379

@Rick-Anderson
Copy link
Contributor Author

@sammychinedu2ky are you able to write a sample?

@sammychinedu2ky
Copy link
Contributor

@sammychinedu2ky are you able to write a sample?

Yh I was able to write one. And it has been merged

@Rick-Anderson
Copy link
Contributor Author

Yh I was able to write one. And it has been merged
@sammychinedu2ky remind me where the code is at.

@sammychinedu2ky
Copy link
Contributor

Yh I was able to write one. And it has been merged
@sammychinedu2ky remind me where the code is at.

#80

@Rick-Anderson
Copy link
Contributor Author

Fixed in #80

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants