Fix missing app icons in getUserApps by upgrading to API v2 endpoint

[Copilot]

The getUserApps() method was not returning application icon data because it was using the OneLogin API v1 endpoint (/api/1/users/{id}/apps), which doesn't include icon information in the response. This issue affected developers who needed to display app icons in their applications.

Changes Made

Updated API endpoint: Changed GET_APPS_FOR_USER_URL constant from API v1 to v2:

// Before
const GET_APPS_FOR_USER_URL = "https://api.%s.onelogin.com/api/1/users/%s/apps";

// After  
const GET_APPS_FOR_USER_URL = "https://api.%s.onelogin.com/api/2/users/%s/apps";

Enhanced App model: Added safe handling for the icon field to prevent PHP warnings:

// Before
$this->icon = $data->icon;

// After
$this->icon = isset($data->icon) ? $data->icon : null;

Impact

• ✅ Icon data now included: Apps returned by getUserApps() now include icon URLs when available
• ✅ Backward compatible: Apps without icon data safely get null for the icon field (no PHP warnings)
• ✅ No breaking changes: Existing code using getUserApps() continues to work unchanged
• ✅ Minimal changes: Only 2 lines changed across 2 files

Example Usage

$client = new OneLoginClient($clientId, $clientSecret);
$apps = $client->getUserApps($userId);

foreach ($apps as $app) {
    echo "App: " . $app->name . "\n";
    if ($app->icon) {
        echo "Icon: " . $app->icon . "\n"; // Now includes icon URL!
    }
}

The API v2 endpoint provides the same response structure as v1 but includes the missing icon data, resolving the issue described in the original problem statement.

Fixes #32.

---

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.