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

change version.oh to h_version.txt, update getinfo for v3 fixes #69

Open
wants to merge 15 commits into
base: revision
Choose a base branch
from

Conversation

strawbberrys
Copy link
Contributor

this is due to synapse 3.0 having a whitelist on file types. the name isn't very good but can be changed.

this is due to synapse 3.0 having a whitelist on file types. the name isn't very good but can be changed.
@strawbberrys strawbberrys changed the title change version.oh to hydroxide_version.txt change version.oh to h_version.txt Apr 15, 2022
@strawbberrys strawbberrys changed the title change version.oh to h_version.txt change version.oh to h_version.txt, update getinfo for v3 fixes May 16, 2022
@Upbolt
Copy link
Owner

Upbolt commented May 27, 2022

I remember Louka being aware of esoteric file extensions and speaking on the matter so I don't know why they would add a whitelist, but if this is the case, I'll talk to 3ds and Louka about whitelisting the "oh" extension. Until then though, I'll leave this open so you can remove the name changes in your commits & only leave the getinfo fix

@Upbolt Upbolt closed this May 27, 2022
@Upbolt Upbolt reopened this May 27, 2022
@Upbolt
Copy link
Owner

Upbolt commented May 27, 2022

oops lol

@strawbberrys
Copy link
Contributor Author

strawbberrys commented Aug 22, 2022

this probably will never be changed. i feel like it would be easier to change the file to .txt instead of trying to get the .oh file whitelisted. not only that but having this fix implemented would allow people to use hydroxide on v3 instead of having to use a fork as no other remote spy has been created yet.

@Upbolt
Copy link
Owner

Upbolt commented Aug 23, 2022

does synapse v3 have any globals we can use instead of calling identifyexecutor? i don't think a majority of freesploits have that func

@strawbberrys
Copy link
Contributor Author

strawbberrys commented Aug 29, 2022

I don't think it has anything besides its new custom functions. I updated it to make sure that identifyexecutor exists before checking.

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.

None yet

2 participants