Fixing a few issues by AcidRaZor · Pull Request #8 · JingoC/WebDavServer

Update Dockerfile:

AcidRaZor

committed

- Instead of using First() or Last(), changed it to use the index instead to get rid of some unnecessary overhead (nitpicky I know)

AcidRaZor

committed

Updated Readme with steps to get this up and running with the Postgres example

AcidRaZor

committed

- Fixed an issue where, when running MKCOL, a PROPFIND check is done first to see if the directory exists, if it didn't gave an error because the database is expected to always return a value

AcidRaZor

committed

- Fixes an issue where an exception is thrown when propertiesList is empty. This happens when the directory or file doesn't exist. As part of the sequence (maybe specific to Windows mapped drives),…

AcidRaZor

committed

- Fixed an issue where an exception was thrown that's unhandled in the WebDAV response when an item doesn't exist. PROPFIND runs before MKCOL so it's important not to throw an exception in this seq…

AcidRaZor

committed

- Changed PROPFIND to be an IActionResult to return 404 NotFound when requesting dumb windows files (Usually through Mapped Drive on Windows machines)

AcidRaZor

committed

- Fixed an issue where the RegEx ignored valid filenames. Important when Windows, by default, creates "New Folder" for you to rename when creatng. This should be cross-platform compatible.

AcidRaZor

committed

- Fixed an issue where, it's checking the parent Id when getting the directory info instead of the actual directory referenced

AcidRaZor

committed

- Align with previous changes for consistency

AcidRaZor

committed

Update ReadMe.md

AcidRaZor

authored

Accidentially inverted the check for HasInvalidChars

AcidRaZor

committed

- Fixes an issue on directory rename where it was referencing the parent instead of the directory being renamed

AcidRaZor

committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Fixing a few issues#8

Fixing a few issues#8
AcidRaZor wants to merge 13 commits intoJingoC:masterfrom
AcidRaZor:master

Commits on Dec 22, 2023

Update Dockerfile:

- Instead of using First() or Last(), changed it to use the index instead to get rid of some unnecessary overhead (nitpicky I know)

Updated Readme with steps to get this up and running with the Postgres example

- Fixed an issue where, when running MKCOL, a PROPFIND check is done first to see if the directory exists, if it didn't gave an error because the database is expected to always return a value

- Fixes an issue where an exception is thrown when propertiesList is empty. This happens when the directory or file doesn't exist. As part of the sequence (maybe specific to Windows mapped drives),…

- Fixed an issue where an exception was thrown that's unhandled in the WebDAV response when an item doesn't exist. PROPFIND runs before MKCOL so it's important not to throw an exception in this seq…

- Changed PROPFIND to be an IActionResult to return 404 NotFound when requesting dumb windows files (Usually through Mapped Drive on Windows machines)

- Fixed an issue where the RegEx ignored valid filenames. Important when Windows, by default, creates "New Folder" for you to rename when creatng. This should be cross-platform compatible.

- Fixed an issue where, it's checking the parent Id when getting the directory info instead of the actual directory referenced

- Align with previous changes for consistency

Update ReadMe.md

Accidentially inverted the check for HasInvalidChars

- Fixes an issue on directory rename where it was referencing the parent instead of the directory being renamed