nodeSolidServer / nodeSolidServer/node-solid-server
Enable WebID-TLS integration tests (fix self-signed cert verification loop)
还没有人认领这个 Issue。
- 主要语言
- JavaScript
- 星标
- 1.8k
- 派生
- 308
- PR 合并指标
- 30 天内没有已合并 PR
描述
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
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
从 test/integration/acl-tls-test.mjs 开始,检查第 67、134、571、616 和 946 行的五处 describe.skip。然后阅读 .github/workflows/ci.yml,确认 CI 如何配置主机。完成的标准是 WebID-TLS 集成测试在 CI 中运行,并且 tim.localhost 和 nicola.localhost 都解析到 127.0.0.1。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- github-actions, javascript
- 领域
- ci-cd, testing
- Issue 类型
- 缺陷
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 活跃度
- 停滞
- 描述清晰度
- 描述清楚
- 新手友好度
- 65/100