getPod() assumes subdomain pattern, breaks path-based pods

Offen

#651 erstellt am 29.12.2025 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 48/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: typescript
Bereich: authentication

Rechercherichtung

Beginnen Sie mit getPod() in src/utils/headerFooterHelpers.ts und untersuchen Sie, wie authn.currentUser(), WebID-Daten und storage in den umgebenden Hilfsfunktionen verfügbar sind. Überprüfen Sie das Verhalten mit einem pfadbasierten Pod wie http://server.com/alice/ und behalten Sie den bestehenden ursprungsbasierten Fallback für Subdomain-Pods bei.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Problem

The getPod() function in src/utils/headerFooterHelpers.ts assumes pods are at the origin (subdomain pattern), which breaks path-based pod servers.

Current code (line 17-20):

export function getPod (): NamedNode {
  // @@ TODO: This is given that mashlib runs on NSS - might need to change when we want it to run on other Pod servers
  return sym(document.location.origin).site()
}

The TODO comment acknowledges this limitation.

Impact

When using a path-based pod server (e.g., http://server.com/alice/), after login:

WebID: http://server.com/alice/profile/card#me ✓
getPod() returns: http://server.com/ ✗
Profile guess becomes: http://server.com/profile/card#me ✗
Should be: http://server.com/alice/profile/card#me ✓

This causes 404 errors when mashlib tries to load the user's profile.

Affected Servers

Any path-based pod server
CSS (Community Solid Server) supports both subdomain and path patterns
JSS (JavaScript Solid Server) uses path pattern

Suggested Fix

Derive the pod root from the logged-in user's WebID rather than assuming document.location.origin:

export function getPod (): NamedNode {
  const user = authn.currentUser()
  if (user) {
    // Extract pod root from WebID's pim:storage or by convention
    const storage = getStorage(user)
    if (storage) return storage
  }
  // Fallback to origin-based (subdomain pattern)
  return sym(document.location.origin).site()
}

Or use the pim:storage triple from the user's profile which explicitly declares the pod root.

Environment

solid-ui version: latest (via mashlib CDN)
Server: JSS (JavaScript Solid Server) with path-based pods

Vorherrschende Sprache: TypeScript
Sterne: 153
Forks: 46
Ø Merge: 1 T. 8 Std.
Gemergte PRs (30 T.): 7

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus SolidOS/solid-ui

Empty field upon login is not user friendly Offen

enhancement

SolidOS/solid-ui#866 · 1 zugewiesene Person · Erstellt am 31.8.2026
When uploadFiles fails on PUT the catch error says only FAILS on PUT Offen

enhancement

SolidOS/solid-ui#861 · 1 zugewiesene Person · Erstellt am 24.8.2026
Vite warning Offen

enhancement

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 64/100

SolidOS/solid-ui#860 · Erstellt am 22.8.2026
Menu needs configuration for styling Offen

enhancement

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

SolidOS/solid-ui#856 · Erstellt am 15.8.2026
Move file explorer group action buttons Offen

enhancement

SolidOS/solid-ui#848 · 1 zugewiesene Person · Erstellt am 27.7.2026

Alle Issues in SolidOS/solid-ui