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

Session Hijacker: only 2nd level domain shown #21

Closed
dobrik opened this issue Nov 11, 2014 · 5 comments · Fixed by #485
Closed

Session Hijacker: only 2nd level domain shown #21

dobrik opened this issue Nov 11, 2014 · 5 comments · Fixed by #485
Labels
MITM

Comments

@dobrik
Copy link

dobrik commented Nov 11, 2014

hi! On Session Hijacker do not working with domain 2nd level.I will see "gorod.dp.ua" as "dp.ua".
@tux-mind
Copy link
Member

@dobrik MITM function are the same of dSploit.
i will fix it when i'll write MITM functions in the core.

thanks for the report.

@tux-mind tux-mind changed the title Session Hijacker bug Session Hijacker: only 2nd level domain shown Nov 11, 2014
@tux-mind tux-mind added the MITM label Nov 11, 2014
@dobrik
Copy link
Author

dobrik commented Nov 11, 2014

See the screensot at the attachment. Site jacked e36club.com.ua
11 нояб. 2014 22:14 пользователь "tux-mind" [email protected]
написал:

@dobrik https://github.com/dobrik MITM function are the same of dSploit.
i will fix it when i'll write MITM functions in the core.

thanks for the report.

—
Reply to this email directly or view it on GitHub
#21 (comment).

@tux-mind
Copy link
Member

hum... i think that gihub drop sent email attachments, you have to send it on the github web interface.

thanks for your time 😊

@dobrik
Copy link
Author

dobrik commented Nov 12, 2014

screenshot_2014-11-11-22-21-19

@Interneedus
Copy link
Contributor

@dobrik Use SD Maid instead of Clean Master. (Sorry for off topic post)

gustavo-iniguez-goya pushed a commit to gustavo-iniguez-goya/android that referenced this issue Nov 1, 2015
@gainan
parse TopLevelDomains correctly.
7f8ea0a 
put out of the loop the check for adding dots to the domain.

fixes cSploit#21.
@gustavo-iniguez-goya gustavo-iniguez-goya mentioned this issue Nov 1, 2015
Merged
tux-mind pushed a commit that referenced this issue Mar 28, 2016
@gainan @tux-mind
parse TopLevelDomains correctly.
7d983d5 
put out of the loop the check for adding dots to the domain.

fixes #21.
@michael86 michael86 mentioned this issue May 5, 2017
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
MITM
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Parse TopLevelDomains correctly. gustavo-iniguez-goya/android
3 participants
@tux-mind @dobrik @Interneedus