nodeSolidServer / nodeSolidServer/node-solid-server
Enable WebID-TLS integration tests (fix self-signed cert verification loop)
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 1.8k
- Forks
- 308
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
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:modulusin profile matches cert ✅
Why It Matters
- README still advertises
--auth tlssupport - 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
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
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.
Bewertung
- Tech-Stack
- github-actions, javascript
- Bereich
- ci-cd, testing
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 65/100