Skip to content

docs: sync release, frontend, and GeoIP guidance - #336

Merged
XiNian-dada merged 9 commits into
mainfrom
claude/g1-docs
Jul 26, 2026
Merged

docs: sync release, frontend, and GeoIP guidance#336
XiNian-dada merged 9 commits into
mainfrom
claude/g1-docs

Conversation

@XiNian-dada

Copy link
Copy Markdown
Owner

Summary

  • backfill CHANGELOG.md from v2.2.7 through v3.0.2-rc.2
  • replace the obsolete Stage 0 frontend README with the current Vue architecture, routes, state flow, commands, tests, and embedded-build workflow
  • document NODELITE_BUILD_VERSION as the official tag-derived version source and workspace 0.1.0 as the local fallback
  • disclose the default online ipwhois GeoIP lookup, the public connection IP sent to https://ipwho.is, and offline/disabled alternatives

Verification

  • git diff --check NodeLite/main...HEAD
  • pnpm --dir nodelite-server/web lint
  • pnpm --dir nodelite-server/web typecheck
  • pnpm --dir nodelite-server/web test (81 files, 490 tests)
  • cargo test -p nodelite-proto parses_server_config_with_defaults
  • cargo test --workspace
  • cargo clippy --all-targets -- -D warnings
  • verified config/server.example.toml changes are comments only
  • verified GeoIP defaults and implementation are unchanged

Closes #318
Closes #310

XiNian-dada and others added 4 commits July 27, 2026 02:12
Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Claude <noreply@anthropic.com>

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b72cdce790

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread README.md Outdated
- 敏感配置优先通过服务端文件、CLI 和受保护设置入口修改。
- `/metrics` 与面板共用只读认证,适合接入 Prometheus。
- 历史图用于展示基础趋势,不是每条 `metrics` 上报的完整归档。
- GeoIP 默认启用在线 `ipwhois` provider。Server 会把解析出的 Agent 公网连接 IP 发送到第三方接口 `https://ipwho.is`,用于查询国家/地区以及可用的城市和坐标信息;局域网、回环和文档保留地址会在本地标记为 LAN,不发起该在线查询。

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Disclose dashboard-user IP lookups

With the default ipwhois provider, the disclosure is not limited to Agent addresses: successful Basic Auth requests call state.geoip.lookup(meta.client_ip) in nodelite-server/src/handlers/auth_routes/middleware.rs, and successful TOTP logins do the same in two_factor.rs, causing public dashboard-user IPs to be sent to ipwho.is as well. Operators relying on this privacy notice may therefore leave the feature enabled without realizing that viewer/admin addresses are disclosed; update this notice and the equivalent English README and example-config comments to describe both Agent and login-client lookups.

Useful? React with 👍 / 👎.

@XiNian-dada
XiNian-dada merged commit f80676b into main Jul 26, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[P3][docs] 同步 v3 CHANGELOG、前端 README 与版本来源说明 [P3][docs] 补充默认在线 GeoIP 行为与第三方查询说明

1 participant