Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[dcom] fix dcom disconnect issues #1

Closed
wants to merge 2 commits into from
Closed

[dcom] fix dcom disconnect issues #1

wants to merge 2 commits into from

Conversation

XiaoliChan
Copy link

@XiaoliChan
Copy link
Author

@mpgn This patch is necessary for CME, it fixes the dcom.disconnect() function in CME everywhere.

@XiaoliChan
Copy link
Author

XiaoliChan commented Sep 4, 2023 •
edited
Loading

An idea came to me about the dcom timeout after I read the dcomrt.py again and again.

I think we can define a global variable like dcom_timeout = 300, and we maybe can change it like this

DCOMConnection.dcom_timeout = 10
dcom = DCOMConnection(bla bla bla)

@XiaoliChan
Copy link
Author

Confirm dcom.set_connect_timeout is work

@XiaoliChan
Copy link
Author

Before, it will be hanging long time
image

with dcom.set_connect_timeout(10)
image

This was referenced Sep 5, 2023
Closed
Open
@XiaoliChan XiaoliChan closed this by deleting the head repository Sep 5, 2023
@XiaoliChan XiaoliChan mentioned this pull request Sep 6, 2023
Merged
mpgn pushed a commit that referenced this pull request Jun 18, 2024
@zblurx @dru1d-foofus
Implement MS-GKDI and LAPSv2 password extraction (fortra#1556)
7e25245 
* add MS-GKDI

* update gkdi

* Update getLAPSv2Password.py

* Added GetLAPSPassword to examples. (#1)

* Added GetLAPSPassword to examples.

* update GetLapsPassword

* Add LAPSv2 column

---------

Co-authored-by: dru1d <[email protected]>
Co-authored-by: zblurx <[email protected]>

* update GetLAPSPassword.py

* Added tab delimited outputfile option (#2)

Co-authored-by: dru1d <[email protected]>

* fix requirements.txt

---------

Co-authored-by: Tyler <[email protected]>
Co-authored-by: dru1d <[email protected]>
Co-authored-by: = <=>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@XiaoliChan