Error when starting Tor.exe about walletwasabi HOT 12 CLOSED

Looks fine for me from the first glance. Could you try the release binary so we can rule out if it's your environment or not? (Check in task manager if tor is running after hw is launched)

I will take a closer look it later when I get to the computer.

from walletwasabi.

In the meantime I figured out what's the problem. You launched the API from Visual Studio, but it's a little more complicated, because .NET Core has no built in GUI support, so I had to use Electron.

Which goes like this:
As in the "Building from source code/ Step by step" section you do this:
First you build the HiddenWallet.API, then you publish it to bin/dist/current-target location.
Then you copy Tor and run HiddenWallet.GUI with npm start.

The HiddenWallet.GUI is Electron application, so it will launch the api from bin/dist/current-target and the api will correctly find the Tor, since you copied there.

However if you try to launch it from Visual Studio, it builds the HiddenWallet.API to bin/Debug or bin/Release, depending on your settings. So tor must be copied there somewhere.

What still worries/confuses me is you didn't succeed by using torPath = @"C:\Users\Bijan\HiddenWallet\HiddenWallet\HiddenWallet.API\bin\dist\current - target\tor\Tor\tor.exe";, which I suspect is a separate issue. Maybe you specified the path wrongly? Is File.Exists() true?

from walletwasabi.

Adam,
Yes I am using VS 2017 RC. I reinstalled the project again as I had made many changes. Now I can confirm that if I use the file path to the tor.exe, it does launch it. Now I decided to test the other option of using relative path so I got the location of the executing assembly using this line: string p1 = System.IO.Path.GetDirectoryName(System.Reflection.Assembly.GetEntryAssembly().Location);
and I placed the Tor in that folder, and it did not work! So I figured VS must be looking in another folder and that folder can be found by right clicking the project and choosing open folder in file explorer. I placed the Tor folder in this directory: C:\Users\Bijan\HiddenWallet\HiddenWallet\HiddenWallet.API
So the path to tor.exe on my computer became: C:\Users\Bijan\HiddenWallet\HiddenWallet\HiddenWallet.API\tor\tor\tor.exe
And the error was resolved.
Thanks - Bijan

from walletwasabi.

Awesome!

from walletwasabi.

Hi I'm having the same issue is there a easy fix for this. I'm a bit lost

from walletwasabi.

Moved to #27

from walletwasabi.

from walletwasabi.

FYI: The latest changes alters UI as shown below. The green bar on the
bottom blocks Decrypt Caption. It's cosmetic, but makes your UI much nicer.

You messed up the image uploading. Could you upload to imgur maybe? (Quick and easy).
I'll fix it, once I figure out what's the problem:)

And a suggestions: Why not make the window resizable. I changed it on mine.

This is not a good idea. This was my first attempt at web development (Electron is creating desktop app with web development tools) and I was in the mentality of "have something". Therefore resizing stuff I expect to break a lot of things.
In the future (at the stable release phase in the roadmap) I either rewrite the GUI in TypeScript (right now it's pure JavaScript) or Microsoft already came out with GUI support for .NET Core.

Question: Do you prefer me logging it in GitHub or email when small stuff?

GitHub better. Don't be rest to open issues.

from walletwasabi.

from walletwasabi.

from walletwasabi.

from walletwasabi.

@Bijan-Ghofranian You are using GitHub through email. This apparently has the unfortunate side effect of nobody can see the attachments. Could you upload the image to http://imgur.com/, since I am not able to discover the UI bug based on your explanation. Maybe it is environment dependent?

from walletwasabi.