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.

Sign up for GitHub

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

Jump to bottom

Define behavior of PathUtils.ResolveAppRelative and variants #102

Open
lilith opened this issue Sep 27, 2014 · 3 comments
Open

Define behavior of PathUtils.ResolveAppRelative and variants #102

lilith opened this issue Sep 27, 2014 · 3 comments
Labels
ready
Milestone
Backlog

Comments

@lilith
Copy link
Member

lilith commented Sep 27, 2014

ResolveAppRelative and ResolveAppRelativeAssumeAppRelative currently have undefined behavior in the absence of an HttpEnvironment. In the develop branch, one resolves to a physical path, while the other remains a virtual path.
@GravitasExtra
Copy link
Contributor

Good point. I have updated ResolveAppRelative to be virtual.

@lilith
Copy link
Member Author

lilith commented Oct 29, 2014

We should add unit tests for this (and other PathUtils methods)

@lilith
Copy link
Member Author

lilith commented Nov 11, 2014

Both with and without an HttpContext...

@lilith lilith added the ready label Nov 23, 2014
@lilith lilith modified the milestone: Backlog Dec 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
ready
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
2 participants
@lilith @GravitasExtra