Enable WebID-TLS integration tests (fix self-signed cert verification loop)

Offen Anfängerfreundlich

#1,841 erstellt am 08.1.2026 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 65/100

Issue-Typ: Bug
Klarheit: Klar beschrieben
Aktivitätsstatus: Veraltet
Tech-Stack: github-actions, javascript
Bereich: ci-cd, testing

Rechercherichtung

Beginne mit test/integration/acl-tls-test.mjs und untersuche die fünf describe.skip-Stellen in den Zeilen 67, 134, 571, 616 und 946. Lies anschließend .github/workflows/ci.yml und überprüfe, wie CI Hosts konfiguriert. Erledigt ist die Aufgabe, wenn die WebID-TLS-Integrationstests in CI mit tim.localhost und nicola.localhost ausgeführt werden und beide zu 127.0.0.1 aufgelöst werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Summary

In October 2019, WebID-TLS tests were disabled with describe.skip() (commit 778095ad). The authentication code still works - just the tests are skipped and CI lacks DNS setup.

Fix size: ~10 lines changed.

The Bug

// Added Oct 29, 2019 by jaxoncreed
describe.skip('ACL with WebID+TLS', function () {  // <-- this .skip

The comment says "TLS is currently broken" but the code isn't broken - the tests just need tim.localhost to resolve to 127.0.0.1.

The Fix

1. Remove 5x .skip from test/integration/acl-tls-test.mjs:

Line 67, 134, 571, 616, 946

2. Add 4 lines to CI (.github/workflows/ci.yml):

- name: Setup hosts for TLS tests
  run: |
    echo "127.0.0.1 tim.localhost" | sudo tee -a /etc/hosts
    echo "127.0.0.1 nicola.localhost" | sudo tee -a /etc/hosts

Done.

Verification

The code works today:

Server starts with --auth tls ✅
Responds to requests ✅
Test certificates match test profiles ✅
cert:modulus in profile matches cert ✅

Why It Matters

README still advertises --auth tls support
Solid spec requires OIDC, doesn't forbid TLS
Server-to-server, CLI, IoT use cases need non-browser auth
W3C LWS working group discussing enterprise mTLS needs (LWS-UCS #40)

References

Commit that introduced skip: 778095ad (Oct 29, 2019)
PR where TimBL approved TLS+OIDC coexistence: #650

Vorherrschende Sprache: JavaScript
Sterne: 1.8k
Forks: 308
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Beitragsleitfaden öffnen

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 nodeSolidServer/node-solid-server

Disclaimer in README.md about non-production readiness Offen

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 72/100

nodeSolidServer/node-solid-server#1848 · 2 Kommentare · 1 Reaktion · Erstellt am 07.2.2026
Delete account must point to POD provider Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100

nodeSolidServer/node-solid-server#1147 · Erstellt am 15.3.2019
remove bootstrap dependency Offen

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

nodeSolidServer/node-solid-server#1867 · Erstellt am 09.5.2026
Align NSS with LWS 1.0 Authentication Suite Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 45/100

nodeSolidServer/node-solid-server#1853 · Erstellt am 23.4.2026
npm: Hash-based Publishing Breaking Semver Convention Offen

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

nodeSolidServer/node-solid-server#1852 · 1 Kommentar · Erstellt am 22.3.2026

Alle Issues in nodeSolidServer/node-solid-server