Skip to content

Source handler check#33

Merged
robamu merged 7 commits intous-irs:mainfrom
sgaisser:source_handler_check
Dec 19, 2024
Merged

Source handler check#33
robamu merged 7 commits intous-irs:mainfrom
sgaisser:source_handler_check

Conversation

@sgaisser
Copy link
Copy Markdown
Contributor

Fixed

  • Source Handler checked the file path directly instead of using the vfs

Added

  • Restricted file handler which restricts paths to a certain path and children of it.
  • More tests for filestores, still not all branches covered and CRC modular might not work as expected for special VFS.

@sgaisser sgaisser marked this pull request as draft December 18, 2024 17:56
@sgaisser sgaisser marked this pull request as ready for review December 19, 2024 10:01
@robamu robamu added this to the vNext milestone Dec 19, 2024
@robamu robamu merged commit 994c066 into us-irs:main Dec 19, 2024
@robamu
Copy link
Copy Markdown
Contributor

robamu commented Dec 19, 2024

thanks!

@sgaisser sgaisser deleted the source_handler_check branch December 19, 2024 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants