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

Ouverte Adaptée aux débutants
#1,841 2 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
65/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
À l'abandon
Stack technique
github-actions, javascript
Domaine
ci-cd, testing

Piste de recherche

Commencez par test/integration/acl-tls-test.mjs et examinez les cinq emplacements de describe.skip aux lignes 67, 134, 571, 616 et 946. Lisez ensuite .github/workflows/ci.yml et vérifiez comment CI configure les hosts. Le travail est terminé lorsque les tests d’intégration WebID-TLS s’exécutent dans CI avec tim.localhost et nicola.localhost résolvant vers 127.0.0.1.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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
Langage dominant
JavaScript
Étoiles
1.8k
Forks
308
Aucune PR mergée en 30 j

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de nodeSolidServer/node-solid-server

Toutes les issues de nodeSolidServer/node-solid-server

Issues similaires

Plus d'issues JavaScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.