Skip to content
This repository was archived by the owner on Apr 20, 2023. It is now read-only.

Conversation

@mlorbetske
Copy link

Per @danroth27, if the user sets the environment variable in a launch profile for ASPNETCORE_URLS, it should override the value of ApplicationUrl. Added tests to check that behavior & verify that ApplicationUrl is being applied in general.

@mlorbetske mlorbetske requested a review from livarcocc March 22, 2018 04:21
@mlorbetske
Copy link
Author

@dotnet-bot test Windows_NT x64 Release Build please
@dotnet-bot test Windows_NT x86 Debug Build please
@dotnet-bot test Windows_NT_ES x64 Debug Build please

@livarcocc
Copy link

@mlorbetske the failures seem to be legit on windows: the test that is failing is AppWithApplicationUrlInLaunchSettings.

@mlorbetske
Copy link
Author

@livarcocc it's strange, the error indicates that the runtimeconfig is not valid. I've removed the RIDs from the project file - hopefully that takes care of it

@livarcocc
Copy link

@mlorbetske apparently... it did not.

@mlorbetske
Copy link
Author

I think i see, looks like it might be a MAX_PATH thing. Will try a shorter test project name.

@livarcocc
Copy link

@MattGertz TELL

@livarcocc livarcocc added this to the 2.1.3xx milestone Mar 23, 2018
@livarcocc livarcocc merged commit 1409844 into dotnet:release/2.1.3xx Mar 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants