impacket icon indicating copy to clipboard operation
impacket copied to clipboard

Install without examples

Open arossert opened this issue 3 years ago • 12 comments

I'm using this great module in my app and noticed that some anti-virus reports on smbrelayx.py as a CVE. Is there a way to install this without the examples?

Currently, I manually delete this file after the installation.

arossert avatar Jul 30 '20 05:07 arossert

Let me add to this: mimikatz.py will block the installation of the package on my company machine and alert the whole company!!!

Please make the examples optional!

logopk avatar Sep 16 '20 08:09 logopk

This should definitely be done, as this is also a show stopper at my company at the moment. I am willing to provide a patch/PR if support is needed in this regard.

mcpat avatar Feb 01 '21 14:02 mcpat

An idea could be to make a "no_examples" (or at the contrary "with_examples") extra in the package, just like this for the requests package.

So we could either do a "pip install impacket" (full install) or "pip install impacket[no_examples]" (without examples).

maaaaz avatar Mar 07 '21 12:03 maaaaz

Hi.

IMO, if going with the extras model, the impacket library should be the base, and examples should be the extra. I think this aligns more with the packaging norms for handling additional scripts.

Alternately, you could make two separate packages/repo: impacket and impacket-examples (where the latter installs the former anyway). This is probably easier for the average user to consume, since (correct me if I'm wrong) most people tend to use impacket only for the examples anyway.

daddycocoaman avatar May 26 '21 23:05 daddycocoaman

Hello team, while most of the time impacket is used for offensive work and av is disabled, recently I was working with python and doing some work (but not really offensive) and wanted to share to code with my collegue but they cannot install and test my script because of the av and they are not offensive guys and cannot understand why they need to disabled an av to use a lib.

Took me a lot of time a refactor the code with another lib and it was a bit sad since the primary goal of impacket is to be a lib.

My 2 cents.

✌️

mpgn avatar Apr 14 '22 09:04 mpgn

Great thread here In my opinion this repo should be split Impacket : library, no examples Impacket-examples : a separate repo, loaded as submodule here in the examples folder

This would be the perfect solution as the structure wouldn't change that much, the library could be pulled and installed as usual, and the examples' life cycle would be separate and still referenced in the main repo

I can help implement that if needed, but I really think Impacket and its users would benefit from all this

ShutdownRepo avatar Apr 20 '22 18:04 ShutdownRepo

Hi,

We just run into the exact same situation as @arossert. Is there any chance that separation between library and examples will come true?

By the way, the README documentation specifies that python3 -m pip install impacket do not install example scrpits, but it looks like it does.

DavidLevayer avatar Jul 08 '22 12:07 DavidLevayer

Hello, Any news about this topic ? How did you guys get around that problem ?

little-kawa avatar Aug 24 '22 12:08 little-kawa

With Impacket under a new org name, maybe this could be implemented? 👀

daddycocoaman avatar Mar 13 '23 15:03 daddycocoaman

Is there anything that can be done to help move this issue along? The current situation is frustrating and the workaround of disabling your AV to install Impacket tends to cause concern and is impossible in some company-managed systems.

FeldrinH avatar Dec 15 '23 19:12 FeldrinH

Is there any news about this?

arossert avatar Apr 30 '24 10:04 arossert

@arossert we had some disscussions we the team about this. We are going to be addressing this after 0.12 release

anadrianmanrique avatar Apr 30 '24 13:04 anadrianmanrique