ConfuserEx icon indicating copy to clipboard operation
ConfuserEx copied to clipboard

Enhancement of the Project

Open debpedrano opened this issue 7 years ago • 30 comments

Greetings PO

I would like to submit a logo design for your project, is there any chance I can contribute ?

debpedrano avatar Jun 23 '18 03:06 debpedrano

Just learned that project is abandoned. I have forked and so far interested to merge all pull requests to it (I need it for my other projects). If you would like to participate we can combine our efforts.

shytikov avatar Jul 09 '18 12:07 shytikov

@shytikov I would love to work with you :) two idea is better than one, so it is not a problem

debpedrano avatar Jul 09 '18 13:07 debpedrano

My plan is to merge all pull-request that are pending, compile the code with most recent dependencies and put it to NuGet.

If everything will went well with that, I would be logical to use renewed logo that you will provide.

shytikov avatar Jul 10 '18 07:07 shytikov

I believe in you, i know you can do those works. Just enjoy working @shytikov , i will work on the logo and banner part.

debpedrano avatar Jul 10 '18 13:07 debpedrano

@shytikov I already started basically what you want to do as well. Checkout my fork. I also merged some of the pull requests already and fixed a few things of my own.

mkaring avatar Jul 11 '18 12:07 mkaring

great, @shytikov someone is helping you to make the project better. awesome work @mkaring !

debpedrano avatar Jul 11 '18 12:07 debpedrano

Superb! I can abandon my fork then and continue work on yours instead!

shytikov avatar Jul 11 '18 14:07 shytikov

@mkaring That´s great. I will keep looking into your fork too.

alexmurari avatar Jul 11 '18 15:07 alexmurari

@shytikov if you have any desired design for the project, feel free to tell me. I have lots of designs on my mind, but your perspective for the design is important.

image

debpedrano avatar Jul 12 '18 09:07 debpedrano

@mkaring I tried Compressor fail to compress file.

[ERROR] Unknown error occurred. Exception: System.Collections.Generic.KeyNotFoundException: The given key was not present in the dictionary. at System.ThrowHelper.ThrowKeyNotFoundException() at System.Collections.Generic.Dictionary2.get_Item(TKey key) at Confuser.Protections.Compress.ExtractPhase.ResourceRecorder.WriterEvent(Object sender, ModuleWriterEventArgs e) at dnlib.DotNet.Writer.EventHandler21.Invoke(Object sender, TEventArgs e) at dnlib.DotNet.Writer.ModuleWriterOptionsBase.RaiseEvent(Object sender, ModuleWriterEventArgs e) at dnlib.DotNet.Writer.ModuleWriterBase.OnWriterEvent(ModuleWriterEvent evt) at dnlib.DotNet.Writer.ModuleWriterBase.Metadata_MetadataEvent(Object sender, MetadataWriterEventArgs e) at dnlib.DotNet.Writer.Metadata.OnMetadataEvent(MetadataEvent evt) at dnlib.DotNet.Writer.Metadata.Create() at dnlib.DotNet.Writer.Metadata.CreateTables() at dnlib.DotNet.Writer.ModuleWriter.WriteImpl() at dnlib.DotNet.Writer.ModuleWriterBase.Write(Stream dest) at dnlib.DotNet.ModuleDef.Write(Stream dest, ModuleWriterOptions options) at Confuser.Core.ConfuserEngine.WriteModule(ConfuserContext context) at Confuser.Core.ProtectionPipeline.ExecuteStage(PipelineStage stage, Action1 func, Func1 targets, ConfuserContext context) at Confuser.Core.ConfuserEngine.RunPipeline(ProtectionPipeline pipeline, ConfuserContext context) at Confuser.Core.ConfuserEngine.RunInternal(ConfuserParameters parameters, CancellationToken token)

bprg avatar Jul 13 '18 07:07 bprg

@bprg does it work with ConfuserEX 1.0.0 ?

I'm revamping the project step after step here : https://github.com/XenocodeRCE/neo-ConfuserEx

XenocodeRCE avatar Jul 13 '18 13:07 XenocodeRCE

does it work with ConfuserEX 1.0.0 ?

Yes works properly, @mkaring ported to new version dnlib that's why it does not work properly.

bprg avatar Jul 13 '18 13:07 bprg

@bprg Ok, I see, thanks for letting me know 👍

There is 0 valid argument as far as obfuscation is concerned to move to the newer version of dnlib however.

XenocodeRCE avatar Jul 13 '18 13:07 XenocodeRCE

The reason I ported to the new version is that the new version of dnlib supports the portable format of the pdb files. The current version does not.

mkaring avatar Jul 13 '18 15:07 mkaring

@bprg I found the possible reason for the problem. Is there any chance you can share the assembly you tried to compress with me? There are multiple reasons why this could go wrong. I want to ensure that I do not miss anything.

mkaring avatar Jul 13 '18 16:07 mkaring

@bprg I fixed the problem based on my tests. Would be great if you could test it again yourself. If the issue is not resolved, it would be nice if you could add the issue to my fork, as the version in this repository is not affected by the issue and I don't want to clutter this issue. :wink:

mkaring avatar Jul 13 '18 17:07 mkaring

@mkaring The issue tab of your repository is not enabled.

alexmurari avatar Jul 13 '18 18:07 alexmurari

@MurariAlex Fixed that.

mkaring avatar Jul 13 '18 20:07 mkaring

@mkaring Thank you, Now works properly.

bprg avatar Jul 14 '18 07:07 bprg

Two forks already by @XenocodeRCE and @mkaring , great but I think it would be better if they were merged. It's more likely that it will survive.

0xd4d avatar Jul 19 '18 12:07 0xd4d

@0xd4d @mkaring fork uses latest dnlib version which I can't bear (sorry). I'll stick with the classic and continue on my side following latest Ki's recommendations.

XenocodeRCE avatar Jul 19 '18 16:07 XenocodeRCE

@0xd4d @XenocodeRCE @mkaring Apart from the portable pdb compatibility, there is any other benefit in using the latest version of dnlib? Maybe bug fixes or a more stable assembly writing process or framework compatibility? If there´s not, maybe @mkaring can push his stuuf into @XenocodeRCE fork without the latest dnlib stuff, I saw that he made some improvements in the analyzers that can be useful.

alexmurari avatar Jul 19 '18 17:07 alexmurari

@MurariAlex Have a look in the git history, the dnlib used by ConfuserEx is at least two years old.

It's more important to try to use just one fork since more forks = more likely each fork will die.

0xd4d avatar Jul 19 '18 17:07 0xd4d

@0xd4d I prefer epistemic diversity 🍷

XenocodeRCE avatar Jul 19 '18 21:07 XenocodeRCE

I am currently working on merging the updates done by @XenocodeRCE into my branch. There are just the few problems with it and I was able to sort most of them out. The last remaining issue is that the compressor does not work correctly anymore. I update the repository once I sorted this issue out. I don't want to push a state that is broken.

mkaring avatar Jul 20 '18 18:07 mkaring

@mkaring current branches works flowlessly for me ?

XenocodeRCE avatar Jul 20 '18 20:07 XenocodeRCE

hello @everyone , is there any possible my logo will be merge on this project ?

debpedrano avatar Jul 21 '18 12:07 debpedrano

@debpedrano if you give authorisation I can use it. But it is not possible on this repo because owner leaved.

XenocodeRCE avatar Jul 21 '18 14:07 XenocodeRCE

@debpedrano I'll gladly use it once I rework the UI. Do you happen to have those logos as SVG images? Would be great to ensure that the UI scales properly on high-res monitors.

mkaring avatar Jul 22 '18 09:07 mkaring

@XenocodeRCE @mkaring sure, you can freely use my logo . just tell me what sizes you need, or maybe the svg file.

debpedrano avatar Jul 24 '18 08:07 debpedrano