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

Configuration File Path Edited #169

Closed
wants to merge 1 commit into from
Closed

Configuration File Path Edited #169

wants to merge 1 commit into from

Conversation

hbergun
Copy link

@hbergun hbergun commented Mar 20, 2019

Configuration file path does not work as in documentation.

@nblumhardt
Copy link
Member

Thanks for sending the PR. The correct path here will depend on the app that's being written (not all paths have bin in them); the documentation all seems to use Directory.GetCurrentDirectory() so we could follow that convention?

    .SetBasePath(Directory.GetCurrentDirectory())

@hbergun
Copy link
Author

hbergun commented Mar 21, 2019

Thanks for sending the PR. The correct path here will depend on the app that's being written (not all paths have bin in them); the documentation all seems to use Directory.GetCurrentDirectory() so we could follow that convention?

    .SetBasePath(Directory.GetCurrentDirectory())

yes that is correct for on kestrel server.if we use Visual Studio(Not:VsCode) and runing on IIS server

Directory.GetCurrentDirectory();

The code line above gives the location of the IIS and this will cause the error

@tsimbalar tsimbalar mentioned this pull request Mar 25, 2019
@skomis-mm
Copy link
Contributor

I think its better to leave current snippet in the readme for the simplicity of console application, thank you for contribution.

@skomis-mm skomis-mm closed this Aug 27, 2021
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

Successfully merging this pull request may close these issues.

3 participants