pylnker icon indicating copy to clipboard operation
pylnker copied to clipboard

pylnker for python 3

Open Erhie opened this issue 5 years ago • 3 comments

here I have the script pylnkerForPython3.py which runs on python 3 pylnkerForPython3.zip

Erhie avatar Jun 05 '19 09:06 Erhie

My fork should support this, thanks to some work from https://github.com/z00nx https://github.com/KillerInstinct/pylnker

KillerInstinct avatar Jun 06 '19 17:06 KillerInstinct

FWIW, the fork from KillerInstinct does not support Python3 afaict, since byte checks are still using strings rather than the proper type, e.g.:

https://github.com/KillerInstinct/pylnker/blob/master/pylnker.py#L319

Can confirm that the pylnkerForPython3.zip uses the correct types.

tlansec avatar Feb 06 '20 15:02 tlansec

Hi @tlansec, thanks for the info. I didn't do the initial Py3 conversion and at the time didn't have a project using Py3 as a test case. I setup some testing locally with a small sample set and put in type fixes with https://github.com/KillerInstinct/pylnker/commit/8ac04a3d6ae409846c443ddf4940910db7cfd439

There's also proper fixes to the timestamp handling, as the "fixes" in the zipped script simply try/except and ignore, eg:

def ms_time_to_unix_str(windows_time):
    time_str = ''
    try:
        unix_time = windows_time / 10000000.0 - 11644473600
        time_str = str(datetime.datetime.fromtimestamp(unix_time))
    except:
        pass
    return time_str

Happy to accept PRs / Issue Requests if you find further bugs, as at this point, I think this repo is no longer maintained.

KillerInstinct avatar Mar 25 '20 12:03 KillerInstinct