Skip to content

pse2john: port script to Python 3 and handle multiple PSE files - #6040

Open
martingalloar wants to merge 2 commits into
openwall:bleeding-jumbofrom
martingalloar:pse2john
Open

martingalloar wants to merge 2 commits into
openwall:bleeding-jumbofrom
martingalloar:pse2john

Conversation

@martingalloar

Copy link
Copy Markdown
  • Port pse2john.py from Python 2 to Python 3
  • Preserve John-compatible hash output
  • Support multiple input files and optional output files
  • Report unsupported PSE files on stderr while continuing processing
  • Handle plain PSE files without aborting the batch

- Port pse2john.py from Python 2 to Python 3
- Preserve John-compatible hash output
- Support multiple input files and optional output files
- Report unsupported PSE files on stderr while continuing processing
- Handle plain PSE files without aborting the batch
@martingalloar martingalloar linked an issue Sep 21, 2026 that may be closed by this pull request
@solardiz
solardiz requested review from exploide and kholia September 22, 2026 16:07
@solardiz

Copy link
Copy Markdown
Member

Thank you! How has this been tested? Do we have suitable sample files in https://github.com/openwall/john-samples?

@martingalloar

Copy link
Copy Markdown
Author

Oh wasn't aware of that repo! I just submitted a separate PR there :)

openwall/john-samples#51

@exploide exploide left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is clearly an improvement. Thanks!

While I left two comments, I think this would be still acceptable as is.

Side note: This is no longer compatible with python2, which is fine for me but I know that @solardiz was in favor of maintaining compatibility as far as possible. However, since this depends on a third party library that might have diverged after the EOL of Python2 (I don't know), this might be an unreasonable effort.

Comment thread run/pse2john.py Outdated
Comment thread run/pse2john.py Outdated
@martingalloar

Copy link
Copy Markdown
Author

Side note: This is no longer compatible with python2, which is fine for me but I know that @solardiz was in favor of maintaining compatibility as far as possible. However, since this depends on a third party library that might have diverged after the EOL of Python2 (I don't know), this might be an unreasonable effort.

Yes, the core pysap library has moved into Python 3 only, and keeping backwards compatibility is not planned, so it will be a big effort to add it just for this script.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update pse2john script in John repo

3 participants