Summary
The StripeYPT plugin includes a test.php debug endpoint that is accessible to any logged-in user, not just administrators. This endpoint processes Stripe webhook-style payloads and triggers subscription operations, including cancellation. Due to a bug in the retrieveSubscriptions() method that cancels subscriptions instead of merely retrieving them, any authenticated user can cancel arbitrary Stripe subscriptions by providing a subscription ID.
Details
At plugin/StripeYPT/test.php:4, the endpoint checks only for a logged-in user, not for admin privileges:
At lines 27-29, the endpoint accepts a JSON payload from the request and processes it through the Stripe metadata handler:
$obj = StripeYPT::getMetadataOrFromSubscription(json_decode($_REQUEST['payload']));
The call chain proceeds as follows:
test.php calls getMetadataOrFromSubscription()
- Which calls
getSubscriptionId() to extract the subscription ID
- Which calls
retrieveSubscriptions() to interact with the Stripe API
At StripeYPT.php:933, the retrieveSubscriptions() method contains a critical bug where it cancels the subscription instead of just retrieving it:
$response = $sub->cancel();
This same bug also affects the production webhook processing path via processSubscriptionIPN(), meaning both the debug endpoint and the live webhook handler can trigger unintended cancellations.
Proof of Concept
-
Log in as any regular (non-admin) user and obtain a session cookie.
-
Send a crafted payload to the test endpoint with a target subscription ID:
curl -b "PHPSESSID=USER_SESSION" \
"https://your-avideo-instance.com/plugin/StripeYPT/test.php" \
-d 'payload={"data":{"object":{"id":"sub_TARGET_SUBSCRIPTION_ID","customer":"cus_CUSTOMER_ID"}}}'
-
The endpoint processes the payload, calls retrieveSubscriptions(), and the subscription is cancelled via the Stripe API.
-
To enumerate subscription IDs, check if the application exposes them through other endpoints or use predictable patterns:
# Check user subscription details if accessible
curl -b "PHPSESSID=USER_SESSION" \
"https://your-avideo-instance.com/plugin/StripeYPT/listSubscriptions.php"
- The Stripe subscription is now cancelled. The affected user loses access to their paid features.
Impact
Any logged-in user can cancel arbitrary Stripe subscriptions belonging to other users. This causes direct financial damage to the platform operator (lost subscription revenue) and service disruption for paying subscribers who lose access to premium features. The debug endpoint should have been removed from production or restricted to admin-only access, and the retrieveSubscriptions() method should retrieve rather than cancel subscriptions.
- CWE-862: Missing Authorization
- Severity: Medium
Recommended Fix
Two changes are needed:
1. Restrict the debug endpoint to admins at plugin/StripeYPT/test.php:4:
// plugin/StripeYPT/test.php:4
if (!User::isAdmin())
Change User::isLogged() to User::isAdmin() so only administrators can access the debug endpoint.
2. Fix the retrieval bug at StripeYPT.php:933:
Remove the $sub->cancel() call from retrieveSubscriptions() so that the function only retrieves subscription data without cancelling it:
// StripeYPT.php:933 - remove the following line:
// $response = $sub->cancel();
The retrieveSubscriptions() method should retrieve subscription information, not cancel subscriptions as a side effect.
Found by aisafe.io
References
Summary
The StripeYPT plugin includes a
test.phpdebug endpoint that is accessible to any logged-in user, not just administrators. This endpoint processes Stripe webhook-style payloads and triggers subscription operations, including cancellation. Due to a bug in theretrieveSubscriptions()method that cancels subscriptions instead of merely retrieving them, any authenticated user can cancel arbitrary Stripe subscriptions by providing a subscription ID.Details
At
plugin/StripeYPT/test.php:4, the endpoint checks only for a logged-in user, not for admin privileges:At lines 27-29, the endpoint accepts a JSON payload from the request and processes it through the Stripe metadata handler:
The call chain proceeds as follows:
test.phpcallsgetMetadataOrFromSubscription()getSubscriptionId()to extract the subscription IDretrieveSubscriptions()to interact with the Stripe APIAt
StripeYPT.php:933, theretrieveSubscriptions()method contains a critical bug where it cancels the subscription instead of just retrieving it:This same bug also affects the production webhook processing path via
processSubscriptionIPN(), meaning both the debug endpoint and the live webhook handler can trigger unintended cancellations.Proof of Concept
Log in as any regular (non-admin) user and obtain a session cookie.
Send a crafted payload to the test endpoint with a target subscription ID:
The endpoint processes the payload, calls
retrieveSubscriptions(), and the subscription is cancelled via the Stripe API.To enumerate subscription IDs, check if the application exposes them through other endpoints or use predictable patterns:
Impact
Any logged-in user can cancel arbitrary Stripe subscriptions belonging to other users. This causes direct financial damage to the platform operator (lost subscription revenue) and service disruption for paying subscribers who lose access to premium features. The debug endpoint should have been removed from production or restricted to admin-only access, and the
retrieveSubscriptions()method should retrieve rather than cancel subscriptions.Recommended Fix
Two changes are needed:
1. Restrict the debug endpoint to admins at
plugin/StripeYPT/test.php:4:Change
User::isLogged()toUser::isAdmin()so only administrators can access the debug endpoint.2. Fix the retrieval bug at
StripeYPT.php:933:Remove the
$sub->cancel()call fromretrieveSubscriptions()so that the function only retrieves subscription data without cancelling it:The
retrieveSubscriptions()method should retrieve subscription information, not cancel subscriptions as a side effect.Found by aisafe.io
References