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.

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

Smidge should do some validation before processing/writing a file #171

Closed
PhyxionNL opened this issue Nov 16, 2022 · 4 comments · Fixed by #177
Closed

Smidge should do some validation before processing/writing a file #171

PhyxionNL opened this issue Nov 16, 2022 · 4 comments · Fixed by #177

Comments

@PhyxionNL
Copy link
Contributor

Example:

System.IO.IOException: The filename, directory name, or volume label syntax is incorrect. : 'C:\Website\data\Smidge\Cache\U\sw7pscd7ms4e'||''
at System.IO.FileSystem.CreateDirectory(System.String fullPath, System.Byte[] securityDescriptor)
at System.IO.Directory.CreateDirectory(System.String path)
at Smidge.Cache.PhysicalFileCacheFileSystem.WriteFileAsync(System.String filePath, System.String contents) at offset 71
at Smidge.FileProcessors.PreProcessManager.<ProcessFileImpl>d__7.MoveNext() at offset 1373
at Smidge.FileProcessors.PreProcessManager.<ProcessFile>d__6.MoveNext() at offset 328
at Smidge.FileProcessors.PreProcessManager.<ProcessAndCacheFileAsync>d__5.MoveNext() at offset 222
at Smidge.Controllers.SmidgeController.<Bundle>d__7.MoveNext() at offset 1360

Another example:
sw7pscd7ms4e%' AND 2*3*8=6*8

@Shazwazza
Copy link
Owner

Have marked as help wanted if you have time to add some validation/tests?

@PhyxionNL
Copy link
Contributor Author

Can you create a new release for this @Shazwazza? Thanks.

@Shazwazza
Copy link
Owner

@PhyxionNL yep, github was down for a bit on friday when i was going to publish, will do now.

@Shazwazza
Copy link
Owner

@PhyxionNL this is published https://github.com/Shazwazza/Smidge/releases/tag/v4.2.1 thanks so much for you help and patience!

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

Successfully merging a pull request may close this issue.

2 participants