Correction of 'XSendFilePath' configuration behavior. #46
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
XSendFileRootPath
- I created a new configuration that allows you to change the root path (this is important for non-standard installations).As the test environment was windows I added the file 'CMakeLists.txt' to work with clion + cmake + vc17 with apache2 lounge.
The main fix is the behavior of the 'XSendFilePath' configuration that was not in accordance with the documentation.
Config:
Root: /var/www
XSendFilePath: /tmp/videos
Given a relative file via header for the module, for example 'file.mp4', the result was always the directory /var/www/file.mp4. XSendFile Path was being ignored because the existence of the file in the root directory was not verified.
The code snippet below solves the problem because if the file does not exist in the root directory, it will be checked in the next directory that is XSendFile.