Skip to content

Update: Fixes, Sendmail & Token ,Browser Tracking (Beta), Logpath Logging , Token-Skeleton #35

Open
H4xl0r wants to merge 29 commits intofrozenjava:masterfrom
H4xl0r:master
Open

Update: Fixes, Sendmail & Token ,Browser Tracking (Beta), Logpath Logging , Token-Skeleton #35
H4xl0r wants to merge 29 commits intofrozenjava:masterfrom
H4xl0r:master

Conversation

@H4xl0r
Copy link
Copy Markdown
Contributor

@H4xl0r H4xl0r commented Jun 2, 2020

Give it a try and tell me
should run on the newest pineapple os

See my readme for more details
stay safe

@frozenjava
Copy link
Copy Markdown
Owner

Hey @H4xl0r,

I definitely see the utility in making it easy for users to do that feature but making it its on portal type seems un-nessacary to me.

I can imagine a usecase where a user wants or needs the features provided by the targeted portal type and also the proposed token portal type - but now they are left having to implement either the targeting or tokening them selves.

Would it be better to bake this feature into basic and target portal types. Then make it easy for the user to implement the client side changes to add it to their portal? If not, can you provide a use case that requires this being its own portal type?

Thanks!

@H4xl0r
Copy link
Copy Markdown
Contributor Author

H4xl0r commented Jun 6, 2020

Sure the skeleton is not really needed ,
it would even be better to include it into the normal one.
Understand the point !

Thats more up to you its your nice work .
the skeleton was more an explaniation how it works for users who wanna dig in like you, to see what i did and whats the idea behind it.
To also easy test it .

The real portal example with tokens can be found (google login) :
https://github.com/H4xl0r/evilportals/tree/master/portals/google-token

so i think the best way would be to impement the most pulls (https api etc) and let the portal token by side and find a solution to include it into the targetet or normal one .

As said its more an example to test and see how it works :-)

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.

2 participants