-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Tracking calls in nusoap #5
Comments
Yes I agree, this can def be removed, thanks for finding and highlighting this |
Did you mean to close this? I press the wrong button here all the time ;-) |
sorry no! |
This is from a while back. Did this get fixed, or is it still waiting for a pull-request? I can pull my finger out and do that if required. |
We have not yet addressed this issue, if you would like to contribute a fix that would be welcome, but it is something we will get around to addressing |
Twitter connector has moved out of custom/modules and into modules/. Require_once was including from custom/modules and crashed the app.
I have been finding a number of places where details of the site are being tracked. If this tracking was open and clear, I would be ignoring it, but it is obfuscated, so in looks it is up to no good and should be removed.
Here is one in includes/nusoap/nusoap.php around line 9315:
The hex codes translate to:
I can do a pull request, but raising it for confirmation first.
The text was updated successfully, but these errors were encountered: