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

Any interest in deobfuscating the code? #17

Open
gsalsero opened this issue Feb 20, 2025 · 4 comments
Open

Any interest in deobfuscating the code? #17

gsalsero opened this issue Feb 20, 2025 · 4 comments

Comments

@gsalsero
Copy link
Contributor

I see the the following files are obfuscated:

  • vsp.php
  • vutil.php
  • vsp-client.php
  • vsp-q3a.php

I have deobfuscated those files. Before submitting these changes, I wanted to see if there is any reason I should keep it private.

Making this change has allowed be to fix a couple of bugs. One has to do with using the ftp feature to download files in a remote directory. After downloading the files, the code did not properly set itself up to actually process those downloaded files.

Another bug I noticed was that aliases were not properly being handled for the ExcessivePlus mod. When attempting to strip out color codes from the name, it assumed xp player names would only use xp-style color codes. In my case I was using vanilla quake 3 color codes. This caused the aliases to be blank.

@evilru
Copy link
Owner

evilru commented Feb 21, 2025

From what I know, the only reason for it to be still obfuscated is that the original sourcecode got lost.

Would you mind to create a pull request?
I hope it will be possible to review it somehow...

cheers

@gsalsero
Copy link
Contributor Author

Sounds good. I'll prepare a PR for the deobfuscation. If that gets approved then I'll create separate PRs for the bugs I mentioned.

@evilru
Copy link
Owner

evilru commented Feb 23, 2025

Thank you for your excellent PR #18 which significantly improved the codebase! 🎉

I've added some minor improvements on top:

  • Basic Security Hardening
  • Logoutput to Docker

I'd like to work on:

  • Project structure cleanup
  • Code formatting
  • Documentation improvements
  • Modern development setup

Please let me know when you're done with your planned changes, so I can do the cleanup work without causing merge conflicts.

Thanks again for your valuable contribution! 🙏

@gsalsero
Copy link
Contributor Author

You're welcome! I'm happy to help.

These are the changes I have lined up. I'll create PRs for each of them:

  • fix alias handling for ExcessivePlusMod when using vanilla color codes
  • fix ftp downloading and processing all files in ftp dir
  • update vsp-client.php, vsp-q3a.psp, vsp.php, and vutil.php with modern 7.4 syntax.

Also, I couldn't connect to XDebug without making changes to launch.json. I'm curious if you've had any trouble. I had to add this to get it to work:

      "pathMappings": {
        "/vsp": "${workspaceFolder}"
      }

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

No branches or pull requests

2 participants