nodeSolidServer / nodeSolidServer/node-solid-server
Enable WebID-TLS integration tests (fix self-signed cert verification loop)
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- JavaScript
- Estrelas
- 1.8k
- Forks
- 308
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Descrição
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
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece por test/integration/acl-tls-test.mjs e inspecione as cinco ocorrências de describe.skip nas linhas 67, 134, 571, 616 e 946. Em seguida, leia .github/workflows/ci.yml e verifique como o CI configura os hosts. Está concluído quando os testes de integração do WebID-TLS forem executados no CI com tim.localhost e nicola.localhost resolvendo para 127.0.0.1.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- github-actions, javascript
- Domínio
- ci-cd, testing
- Tipo de issue
- Bug
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Status de atividade
- Estagnada
- Clareza
- Claramente especificada
- Facilidade para iniciantes
- 65/100