Skip to content

Application Hosted in Sub-Directory Fix#16

Open
brandon-james-dev wants to merge 3 commits intoStefH:mainfrom
brandon-james-dev:main
Open

Application Hosted in Sub-Directory Fix#16
brandon-james-dev wants to merge 3 commits intoStefH:mainfrom
brandon-james-dev:main

Conversation

@brandon-james-dev
Copy link
Contributor

Changed to a relative path for download7up.js to address issues with applications being hosted on a server in a sub directory ("http://host.tld/app/").

…applications being hosted on a server in a sub directory
@StefH StefH added the bug Something isn't working label Oct 27, 2025
@StefH
Copy link
Owner

StefH commented Nov 1, 2025

@brandon-james-dev
Is it possible to test this?
And does the current code still work?

@brandon-james-dev
Copy link
Contributor Author

@brandon-james-dev Is it possible to test this? And does the current code still work?

With .NET 10's release next week, I will work on creating an example project for .NET 10 and also another one for subdirectory hosting. I will wait for the full release of .NET 10 before that.

@StefH
Copy link
Owner

StefH commented Nov 16, 2025

@brandon-james-dev
.NET 10 is released

@brandon-james-dev
Copy link
Contributor Author

@StefH The library works with .NET 10 with no adjustments needed. I updated it with an example and one line to enable it in NuGet in the csproj file. However, there's some I think, configuration, issue with delivering the file on my ASP.NET Core hosted app (for the subdirectory). It will host the file without the hash in the file name, and when it tries to download it, it will not get it, because it is hosted as only download7up.js. I was not able to figure that out today. The configuration I put in the example project is not the same as I am using in the project from my workplace app. We are hosting it on IIS rather than ASP.NET Core in this way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants