SharkCagey / SharkCage

1 stars 3 forks source link

87 cleanup readme and remove unused .md files #99

Closed SailReal closed 6 years ago

DonatJR commented 6 years ago

alright, the powershell script should now copy the config to the public documents folder and create a registry entry

SailReal commented 6 years ago

alright, the powershell script should now copy the config to the public documents folder and create a registry entry

Just recognized (again 😢), that it isn't enough to copy the config, we have also to set the correct ACL's 🙈. Is this even possible with the script?

screen shot 2018-08-08 at 9 51 18 pm
DonatJR commented 6 years ago

@SailReal Can't be, I verified that the access rights of the copied file were the same as the source file. It might be the checked in config still has the old access rights, can you try replacing the config in the "special_configs" folder with a freshly generated version? Maybe the same thing happened with the installer and setting the access rights explicitely on install was also not necessary.

SailReal commented 6 years ago

Hmmmm that's pretty weird...I re cloned the repo with the same results.

Maybe my Windows is blown up.

SailReal commented 6 years ago

I think its done, this PR should be finally mergeable...do you agree @DonatJR?

DonatJR commented 6 years ago

Now I agree, @SailReal ;)

bencikpeter commented 6 years ago

@SailReal @DonatJR Have you guys tried to run configurator installed with this new script? For me it introduced a bug:

SailReal commented 6 years ago

@bencikpeter will check that this evening sounds interesting 😅

DonatJR commented 6 years ago

@bencikpeter yes I have, works without any problems for me on this branch and on develop. Can you try creating a new config for the Configurator (debug version runs on normal desktop) and check if this config works for you? No idea how a simple copy + paste in the script can be responsible for the app not loading in the cage, tough :/

bencikpeter commented 6 years ago

@DonatJR Allright, maybe my system is somehow messed up, I will do a little cleanup and let you know if it helped 😉

SailReal commented 6 years ago

It switches mi to the desktop, labeler loads, but no window appears and when I close the cage, labelel finishes but I stay on a desktop and am forced to restart the machine

Just created a new clone of the repo and have now the same problem like you @bencikpeter.

I'll narrow the problem down.

SailReal commented 6 years ago

I tried a lot of different commits. On my devices it is working until (including) this commit: 109aaa240776c6d0c5903be09ef9813e3f950c7b

After merging db7a14e549c8b64ee6a1d7fb8c01448458e7548b the CageConfigurator isn't starting anymore. While merging #95 I just tested my default config with Firefox and Keepass, not the Configurator. Is it possible that we have a problem with any kind of permissions?

The weirdest thing: until I re cloned the repo everything seems to be fine. After the re clone, I have now on both versions the problem 🤔.

Because it isn't a problem of this PR, I will create a new issue for that: #104