-
Notifications
You must be signed in to change notification settings - Fork 921
BitPorn.eu: Plugin scrapes forum post author UID instead of authenticated user #2213
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
Comments
our index file allow overide , for example, we can edit the following selector instead of use NexusPHP default {
// ...
"selectors": {
"userBaseInfo": {
"page": "/index.php",
"fields": {
"id": {
"selector": "span#curuser a[href*='userdetails.php'][class*='Name']:first",
"attribute": "href",
"filters": [
"query ? query.getQueryString('id'):''"
]
},
"name": {
"selector": "span#curuser a[href*='userdetails.php'][class*='Name']:first"
},
"isLogged": {
"selector": [
"a[href*='logout.php']"
],
"filters": [
"query.length>0"
]
}
}
},
} @tomyangsh can you check and fix this bug ? |
Hi, @Rhilip Thank you for your guidance about overriding the 🚫 Why override is not a safe fix:The For example:
This means a generic selector like: "a[href*='userdetails.php']" …can easily pick the wrong user (e.g. a forum post author), not the authenticated user. On our tracker, the correct user profile link is within a unique element: <div class="user-container" id="userToggle">
<a href="userdetails.php?id=1">Felhasználói Profil</a>
</div>
✅ Proper solution: fix it inside the extension itselfWe’d like to emphasize that overriding the selector locally is not a long-term fix, because:
🔧 Suggestion:Please update the built-in BitPorn config with a more specific and safe selector: "selector": "#userToggle a[href*='userdetails.php']" This will ensure that only the logged-in user is captured by the plugin — not unrelated authors on the page. Let us know if we can help with testing or verification. Thanks again for maintaining PT Plugin Plus — we really appreciate your work! Best regards, |
Thank you so much for dedicating your time to share your valuable suggestions with us. We'll try to fix those incorrect selectors ASAP. However, PTPP is currently a Manifest V2 extension. As a result, we're unable to publish a full update on the Microsoft Edge Add-ons store. The version available in the store is stuck at Our team is now primarily focused on the development of the next Manifest V3 extension pt-plugins/PT-depiler , it will be available in next months and Old PTPP will be removed in store. Rhilip |
Thank you so much for your quick and detailed response! We really appreciate your efforts and totally understand the current limitations with Manifest V2. We'll keep an eye on the pre-release builds and are happy to test any changes or help with feedback — including on the upcoming PT-Depiler extension. Thanks again for your great work on PT Plugin Plus. 🙏 Best regards, |
PT Plugin Plus version:
v1.6.0 (latest release from Edge Extension Store)
PT Plugin Plus installation method:
Market installation (Microsoft Edge Add-ons)
Browser name and version:
Microsoft Edge 134.0.0.0 (Windows 11)
Whether the browser has other plugins installed:
Yes
Is it working properly after disabling other plugins:
Yes, the issue is unrelated to other extensions.
🐞 Problem Description:
We have discovered a critical parsing bug when using PT Plugin Plus with the BitPorn.eu tracker.
After authentication, the plugin displays the wrong user information instead of the authenticated account’s data.
Specifically, the plugin scrapes the first author listed in the “Last 5 Forum Posts” section on
index.php
, instead of reliably identifying the currently logged-in user.As a result, the plugin assigns the wrong UID (and corresponding stats like uploaded/downloaded amounts, bonus, etc.) to the session.
We verified this using a test account (UID: 40179), but the plugin consistently displayed information for other users such as UID 35354 or 24345 — depending on who appeared first in the homepage’s forum section.
Importantly:
🖼️ Related screenshots:
The user "xiansta" originally identified this issue and provided evidence showing that:
These screenshots can be attached if required.
🔁 Reproduce steps:
index.php
).📌 Additional notes:
✅ Suggested improvement:
Please consider updating the user info parser logic to:
/usercp.php
links),Let us know if we can assist further with testing or provide more information. Thank you for your work on the plugin!
The text was updated successfully, but these errors were encountered: