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.

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

Added Dropbox Phishlet #591

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

charlesbel
Copy link
Contributor

Fully working !

@ghost
Copy link

ghost commented Sep 3, 2021

This doesn't work or at least isn't working anymore.

  1. There are obvious JS checks that aren't replaced. The checks don't block login per se but may affect other things.
n.assertDropboxDomain = function (e) {
    var n = document.createElement('a');
    n.href = e;
    var t = n.hostname || window.location.hostname;
    if ( - 1 === t.indexOf('.dropbox.com', t.length - '.dropbox.com'.length)) throw new Error('Cannot send the CSRF token to ' + t)
  }
})),
  1. Importing cookies doesn't result in getting a logged in session. This is because evilginx's cookies have a different sameSite value from what Dropbox is expecting. Real: "sameSite": "no_restriction",, evilginx: "sameSite": null,. This issue may be due to the JS checks which aren't properly cleared, but I don't know for certain.

@charlesbel
Copy link
Contributor Author

Thanks @tomabel
According to what you said, I fixed it and now cookies works well. Some of the things you said weren't true, but you helped me thanks.

@ghost
Copy link

ghost commented Sep 4, 2021

Thanks @tomabel
According to what you said, I fixed it and now cookies works well. Some of the things you said weren't true, but you helped me thanks.

Cheers. I didn't focus too deep into this to test out my claims when commenting. Removed the incorrect details. Glad to hear you got it working.

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.

1 participant