Post a reply

Before posting, please read how to report bug or request support effectively.

Bug reports without an attached log file are usually useless.

Options
Add an Attachment

If you do not want to add an Attachment to your Post, please leave the Fields blank.

(maximum 10 MB; please compress large files; only common media, archive, text and programming file formats are allowed)

Options

Topic review

martin

Thanks.
Julian304

done! See your email box :)
martin

Julian304 wrote:

PS: About the unit tests, So I wrote a fix, and it also includes some unit tests. Would be nice to contribute those tests to WinSCP.

Can you email me the tests?
Julian304

Thanks!

Thanks also for the development version (in the mail). Currently I have created a work around (copied and fixed `Logger.GetAssemblyFilePath` in our code base - refactored it so it was easy to test).

With the RTM release, I'm happy if I could remove the work around in our code. Thanks!

PS: About the unit tests, So I wrote a fix, and it also includes some unit tests. Would be nice to contribute those tests to WinSCP.
martin

Re: .NET loading failed with UNC path - Logger.GetAssemblyFilePath breaks UNC path

Thanks for your report.

This issue has been added to the tracker:
https://winscp.net/tracker/1671

I've fixed the problem. I'm sending you an email with a development version of WinSCP to the address you have used to register on this forum.

Though, the next time, we of course accept pull requests.

Atm, we do not make code of our tests public.
Julian304

.NET loading failed with UNC path - Logger.GetAssemblyFilePath breaks UNC path

We having trouble loading a website from an UNC path.
If checked the code, and the problem is in Logger.GetAssemblyFilePath

- ExeSessionProcess.GetExecutablePath() fails for UNC paths, it's using Logger.GetAssemblyFilePath, and there is a file:// trick inside it.

In case of the UNC path, the Assembly.Codebase of dll is: "file://myUNCFolder/folder/bin/myDll.DLL" - (that's correct)

By removing the file:// prefix in Logger.GetAssemblyFilePath, it will be "myUNCFolder/folder/bin/myDll.DLL" - and that's incorrect.

Are Pull requests accepted to GitHub? It's correct that there aren't any unit tests yet?