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

Pre-Handshake doesn't work and completes the handshake #152

Closed
Mzack9999 opened this issue Jan 11, 2023 · 0 comments · Fixed by #154 or #156
Closed

Pre-Handshake doesn't work and completes the handshake #152

Mzack9999 opened this issue Jan 11, 2023 · 0 comments · Fixed by #154 or #156
Assignees
@Mzack9999
Labels
Priority: High After critical issues are fixed, these should be dealt with before any further issues. Status: Completed Nothing further to be done with this issue. Awaiting to be closed. Type: Bug Inconsistencies or issues which will cause an issue or problem for users or implementors.
Milestone
tlsx v1.0.4

Comments

@Mzack9999
Copy link
Member

tlsx version:

main|dev

Current Behavior:

-pre-handshake/-ps completes the whole TLS handshake without early exit at certificate phase

Expected Behavior:

Early exit at server certificate phase for TLS<1.2

Steps To Reproduce:

$ tlsx -ps -u scanme.sh -json

image

Credits: @PatrikFehrenbach
@Mzack9999 Mzack9999 added Priority: High After critical issues are fixed, these should be dealt with before any further issues. Type: Bug Inconsistencies or issues which will cause an issue or problem for users or implementors. labels Jan 11, 2023
@Mzack9999 Mzack9999 self-assigned this Jan 12, 2023
@Mzack9999 Mzack9999 mentioned this issue Jan 12, 2023
Merged
@Mzack9999 Mzack9999 linked a pull request Jan 12, 2023 that will close this issue
Fixing zcrypto clone bug #154
Merged
@Mzack9999 Mzack9999 added the Status: Completed Nothing further to be done with this issue. Awaiting to be closed. label Jan 13, 2023
@ehsandeep ehsandeep added this to the tlsx v1.0.4 milestone Jan 15, 2023
@ehsandeep ehsandeep linked a pull request Jan 15, 2023 that will close this issue
tlsx v1.0.4 #156
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Mzack9999 Mzack9999

Labels
Priority: High After critical issues are fixed, these should be dealt with before any further issues. Status: Completed Nothing further to be done with this issue. Awaiting to be closed. Type: Bug Inconsistencies or issues which will cause an issue or problem for users or implementors.
Projects
None yet
Milestone
tlsx v1.0.4
Development

Successfully merging a pull request may close this issue.

Fixing zcrypto clone bug projectdiscovery/tlsx
tlsx v1.0.4 projectdiscovery/tlsx
2 participants
@ehsandeep @Mzack9999