Title: Enable WebID-TLS integration tests (fix self-signed cert verification loop) · Issue #1841 · nodeSolidServer/node-solid-server · GitHub
Open Graph Title: Enable WebID-TLS integration tests (fix self-signed cert verification loop) · Issue #1841 · nodeSolidServer/node-solid-server
X Title: Enable WebID-TLS integration tests (fix self-signed cert verification loop) · Issue #1841 · nodeSolidServer/node-solid-server
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,...
Open Graph 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...
X 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...
Opengraph URL: https://github.com/nodeSolidServer/node-solid-server/issues/1841
X: @github
Domain: patch-diff.githubusercontent.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Enable WebID-TLS integration tests (fix self-signed cert verification loop)","articleBody":"## Summary\n\nIn 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.\n\n**Fix size: ~10 lines changed.**\n\n## The Bug\n\n```javascript\n// Added Oct 29, 2019 by jaxoncreed\ndescribe.skip('ACL with WebID+TLS', function () { // \u003c-- this .skip\n```\n\nThe 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`.\n\n## The Fix\n\n**1. Remove 5x `.skip`** from `test/integration/acl-tls-test.mjs`:\n- Line 67, 134, 571, 616, 946\n\n**2. Add 4 lines to CI** (`.github/workflows/ci.yml`):\n```yaml\n- name: Setup hosts for TLS tests\n run: |\n echo \"127.0.0.1 tim.localhost\" | sudo tee -a /etc/hosts\n echo \"127.0.0.1 nicola.localhost\" | sudo tee -a /etc/hosts\n```\n\nDone.\n\n## Verification\n\nThe code works today:\n- Server starts with `--auth tls` ✅\n- Responds to requests ✅\n- Test certificates match test profiles ✅\n- `cert:modulus` in profile matches cert ✅\n\n## Why It Matters\n\n- README still advertises `--auth tls` support\n- Solid spec requires OIDC, doesn't forbid TLS\n- Server-to-server, CLI, IoT use cases need non-browser auth\n- W3C LWS working group discussing enterprise mTLS needs ([LWS-UCS #40](https://github.com/w3c-lws/lws-ucs/issues/40))\n\n## References\n\n- Commit that introduced skip: `778095ad` (Oct 29, 2019)\n- PR where TimBL approved TLS+OIDC coexistence: #650","author":{"url":"https://github.com/melvincarvalho","@type":"Person","name":"melvincarvalho"},"datePublished":"2026-01-08T22:42:50.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/1841/node-solid-server/issues/1841"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:33295824-a8f7-f261-3a89-40e696413125 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | BA88:30CF41:243E9D5:301B812:6971C085 |
| html-safe-nonce | 2323fce79554b5c6e16e20f857f3d03e992f6cfe3460931e3133441cca89b3c3 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCQTg4OjMwQ0Y0MToyNDNFOUQ1OjMwMUI4MTI6Njk3MUMwODUiLCJ2aXNpdG9yX2lkIjoiOTk1ODI3OTI3ODU4OTIxNjA1IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | 7cb6077f9477f16efdd8ce79f6a07411fe05b477873a1f09d83fbec71ab91a52 |
| hovercard-subject-tag | issue:3794712304 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/nodeSolidServer/node-solid-server/1841/issue_layout |
| twitter:image | https://opengraph.githubassets.com/9a87eb3cbf3c18930b1e6d934676b3801dba9dc3cd4d5e636ad74acff2affd01/nodeSolidServer/node-solid-server/issues/1841 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/9a87eb3cbf3c18930b1e6d934676b3801dba9dc3cd4d5e636ad74acff2affd01/nodeSolidServer/node-solid-server/issues/1841 |
| og:image:alt | 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... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | melvincarvalho |
| hostname | github.com |
| expected-hostname | github.com |
| None | c3b677fea7de290a6c0d1c9ccd6c9909b6a527cf8ee079649e09dc78092d24f1 |
| turbo-cache-control | no-preview |
| go-import | github.com/nodeSolidServer/node-solid-server git https://github.com/nodeSolidServer/node-solid-server.git |
| octolytics-dimension-user_id | 104446421 |
| octolytics-dimension-user_login | nodeSolidServer |
| octolytics-dimension-repository_id | 24306004 |
| octolytics-dimension-repository_nwo | nodeSolidServer/node-solid-server |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 24306004 |
| octolytics-dimension-repository_network_root_nwo | nodeSolidServer/node-solid-server |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 29d7fcca69cafb91ec8a596aaf222fd46e9bc934 |
| ui-target | canary-1 |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width