@fleetbase/solid-engine 0.0.3 → 0.0.5

package/ACL_SOLUTION.md

@@ -0,0 +1,72 @@
+# THE REAL ROOT CAUSE - ACL Permissions
+
+## Summary
+
+The 401 errors are NOT due to authentication or token issues. **The root ACL of each pod does not grant write permissions.**
+
+## Key Facts
+
+1. **CSS default root ACL grants only `acl:Read`** to the authenticated user
+2. **Without `acl:Write` or `acl:Append` in the root ACL**, all write operations fail
+3. **WAC-Allow header shows `user="read"`** - confirming no write permissions
+4. **CSS has no built-in UI** for managing ACLs across pods
+5. **Must update ACL programmatically** before first write operation
+
+## The Solution
+
+Before creating any folders or resources, **update the pod's root ACL** to grant write permissions.
+
+### Workflow
+
+1. User logs in via OIDC (get access token + DPoP key)
+2. Determine pod root URL (e.g., `http://solid:3000/test/`)
+3. **Check if ACL grants write access** by inspecting WAC-Allow header
+4. **If no write access, update the root ACL** at `<podRoot>/.acl`
+5. After ACL update, create containers/resources
+
+### ACL Document Format
+
+```turtle
+@prefix acl: <http://www.w3.org/ns/auth/acl#>.
+
+# Full rights for the pod owner (required)
+<#owner>
+    a acl:Authorization;
+    acl:agent <https://solid.example/user#me>;
+    acl:accessTo <https://solid.example/userpod/>;
+    acl:default <https://solid.example/userpod/>;
+    acl:mode acl:Read, acl:Write, acl:Control.
+
+# Append and read rights for the Fleetbase integration
+<#fleetbase>
+    a acl:Authorization;
+    acl:agent <https://fleetbase.com/agent#me>;
+    acl:accessTo <https://solid.example/userpod/>;
+    acl:default <https://solid.example/userpod/>;
+    acl:mode acl:Append, acl:Read.
+```
+
+### Implementation Steps
+
+1. **GET** `<podRoot>` to check WAC-Allow header
+2. If lacks `append`/`write`, prepare ACL Turtle document
+3. **PUT** `<podRoot>/.acl` with DPoP authentication
+4. After successful ACL update, proceed with folder creation
+
+## Why This Matters
+
+- **acl:agent** identifies who gets the permissions (use WebID)
+- **acl:accessTo** applies to the pod root
+- **acl:default** inherits to all descendants
+- **acl:Append** allows creating resources
+- **acl:Write** allows updating existing ones
+
+## For Fleetbase Integration
+
+The integration should:
+1. Check ACL permissions on first access
+2. Prompt user or automatically update ACL
+3. Use `acl:Append` mode (safer than full Write)
+4. Store ACL update status to avoid repeated checks
+
+This is **not a bug in our code** - it's the expected CSS behavior. Every pod needs explicit ACL configuration for write access.

package/CSS_SCOPE_ISSUE.md

@@ -0,0 +1,140 @@
+# CSS Scope Issue - Root Cause Analysis
+
+## The Problem
+
+Access tokens have `"scope": ""` (empty) even though:
+- ✅ Client is registered with `"scope":"openid webid offline_access"`
+- ✅ CSS server supports `["openid", "profile", "offline_access", "webid"]`
+- ✅ Our code requests `['openid', 'webid', 'offline_access']`
+
+## Investigation Findings
+
+### 1. Client Registration (CORRECT)
+
+File: `/data/.internal/idp/adapter/Client/jwRIERi9-RW7LU_42zM3f$.json`
+
+```json
+{
+  "client_id": "jwRIERi9-RW7LU_42zM3f",
+  "client_name": "Fleetbase-v2",
+  "scope": "openid webid offline_access",  ← Registered correctly
+  "dpop_bound_access_tokens": false  ← Should this be true?
+}
+```
+
+### 2. CSS Server Configuration (CORRECT)
+
+File: `/config/config/identity/handler/base/provider-factory.json`
+
+```json
+{
+  "scopes": ["openid", "profile", "offline_access", "webid"],  ← Server supports these
+  "features": {
+    "dPoP": { "enabled": true }
+  }
+}
+```
+
+### 3. Grant Storage (ISSUE FOUND!)
+
+File: `/data/.internal/idp/adapter/Grant/-HGiJLKCUJ7CZcU0ePOWQdw84p_WSNKWp3ajB3Q_mf3$.json`
+
+```json
+{
+  "accountId": "http://solid:3000/test/profile/card#me",
+  "clientId": "jwRIERi9-RW7LU_42zM3f",
+  "openid": {
+    "scope": "openid webid"  ← Only these two! Missing offline_access!
+  }
+}
+```
+
+**Problem:** The grant only includes `"openid webid"`, not `"offline_access"`.
+
+### 4. Access Token (RESULT)
+
+From logs:
+```json
+{
+  "webid": "http://solid:3000/test/profile/card#me",
+  "scope": "",  ← Empty!
+  "client_id": "jwRIERi9-RW7LU_42zM3f"
+}
+```
+
+## Root Cause
+
+The scope is stored in the Grant under the `"openid"` key:
+```json
+"openid": {"scope": "openid webid"}
+```
+
+But CSS might not be extracting it correctly when issuing access tokens, resulting in empty scope.
+
+## Possible Solutions
+
+### Option 1: Fix Grant Scope Storage
+
+CSS might be storing scopes incorrectly. This could be:
+- A CSS bug
+- A configuration issue
+- Expected behavior for certain grant types
+
+### Option 2: Use Different Grant Type
+
+The current grant type might not support scopes properly. Try:
+- Client credentials grant (already tried, same issue)
+- Refresh token flow
+- Different authorization parameters
+
+### Option 3: Manual Scope Injection
+
+Modify CSS code or configuration to ensure scopes are included in access tokens.
+
+### Option 4: Workaround - Use ACL Without Scope
+
+Since the ACL is already correct and grants full permissions, the issue is that CSS requires `webid` scope to authenticate as the WebID.
+
+**Workaround:** Use a different authentication method that doesn't require scope:
+- CSS credential tokens (tried, same scope issue)
+- Direct WebID authentication
+- Bearer tokens without scope validation
+
+## Next Steps
+
+1. **Check if `dpop_bound_access_tokens` should be true**
+   - Current: `false`
+   - Might affect scope handling
+
+2. **Verify authorization request includes scope parameter**
+   - Check if scope is being sent during auth code exchange
+   - Verify it's not being filtered out
+
+3. **Test with explicit scope in token request**
+   - Add scope parameter to token exchange request
+   - See if CSS honors it
+
+4. **Check CSS logs for scope-related warnings**
+   - CSS might be logging why scopes are being dropped
+
+## CSS Configuration Files Analyzed
+
+- `/config/config/oidc.json` - Main OIDC config
+- `/config/config/identity/oidc/default.json` - OIDC handler
+- `/config/config/identity/handler/base/provider-factory.json` - Scope definitions
+- `/config/config/ldp/metadata-writer/writers/www-auth.json` - WWW-Authenticate header
+- `/data/.internal/idp/adapter/Client/*` - Client registrations
+- `/data/.internal/idp/adapter/Grant/*` - Authorization grants
+
+## Conclusion
+
+The issue is NOT with:
+- ❌ Client registration (correct)
+- ❌ Server configuration (correct)
+- ❌ Our code (correct)
+
+The issue IS with:
+- ✅ Grant scope storage (only `"openid webid"`, missing `"offline_access"`)
+- ✅ Access token scope extraction (returns empty instead of grant scopes)
+
+This appears to be a CSS behavior or bug where scopes are not properly propagated from grants to access tokens.

package/HOTFIX_SYNTAX_ERROR.md

@@ -0,0 +1,100 @@
+# Hotfix: PodService.php Syntax Error
+
+## Issue
+After the refactoring, a critical syntax error was introduced in `PodService.php` that broke **all** endpoints, including logout.
+
+## Root Cause
+When adding the new methods `getPodUrlFromWebId()`, `createFolder()`, and `deleteResource()` to PodService.php, they were accidentally placed **outside** the class definition.
+
+**Before Fix (BROKEN):**
+```php
+class PodService
+{
+    // ... existing methods ...
+    
+    private function generateTurtleMetadata(string $name, ?string $description = null): string
+    {
+        // ...
+        return $turtle;
+    }
+} // ← Class closed here at line 743
+
+    // ❌ Methods added OUTSIDE the class - SYNTAX ERROR!
+    public function getPodUrlFromWebId(string $webId): string
+    {
+        // ...
+    }
+    
+    public function createFolder(SolidIdentity $identity, string $folderUrl): bool
+    {
+        // ...
+    }
+    
+    public function deleteResource(SolidIdentity $identity, string $resourceUrl): bool
+    {
+        // ...
+    }
+}
+```
+
+## Error Message
+```
+ParseError: syntax error, unexpected token "public", expecting end of file 
+at /fleetbase/packages/solid/server/src/Services/PodService.php:751
+```
+
+## Fix Applied
+Moved the three methods **inside** the class before the closing brace.
+
+**After Fix (WORKING):**
+```php
+class PodService
+{
+    // ... existing methods ...
+    
+    private function generateTurtleMetadata(string $name, ?string $description = null): string
+    {
+        // ...
+        return $turtle;
+    }
+
+    // ✅ Methods now INSIDE the class
+    public function getPodUrlFromWebId(string $webId): string
+    {
+        // ...
+    }
+    
+    public function createFolder(SolidIdentity $identity, string $folderUrl): bool
+    {
+        // ...
+    }
+    
+    public function deleteResource(SolidIdentity $identity, string $resourceUrl): bool
+    {
+        // ...
+    }
+} // ← Class properly closed at line 852
+```
+
+## Impact
+- **Before:** ALL endpoints returned 500 errors due to class loading failure
+- **After:** All endpoints working normally
+
+## Files Modified
+- `server/src/Services/PodService.php` - Fixed class structure
+
+## Testing
+After this fix:
+1. ✅ Application loads without errors
+2. ✅ Logout endpoint works
+3. ✅ Authentication status endpoint works
+4. ✅ All other endpoints functional
+
+## Prevention
+- Always verify class structure when adding new methods
+- Use IDE with PHP syntax checking
+- Test basic endpoints after structural changes
+- Run `php -l` to check syntax before committing (if PHP CLI available)
+
+## Apology
+This was a careless error on my part during the refactoring. I should have verified the class structure after adding the methods via the `append` action. The methods should have been inserted before the closing brace, not after.