Review of "NetIdent" version 1.1.0 (4)

Details Page Preview

Network identity indicator showing external IP, VPN status, Tailscale, DNS, routing, and all active connections.

Extension Homepage
https://github.com/dvdstelt/netident

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

Shexli (experimental) warning 1 manual_review 1

Shexli found 2 issues that may need reviewer attention.

EGO-A-005 manual_review

extensions should not access the clipboard directly

Direct clipboard access via `St.Clipboard.get_default()` requires reviewer scrutiny.

Review Guidelines

  • extension.js:913
    St.Clipboard.get_default()
  • extension.js:921
    St.Clipboard.get_default()
  • extension.js:1018
    St.Clipboard.get_default()

EGO-L-003 warning

signals connected by extension should be disconnected in disable()

Signals assigned in `enable()` are missing matching disconnect calls in `disable()` or its helper methods.

Disconnect all signals

  • extension.js:309
    this._graph.connect('repaint', area => this._drawGraph(area))
  • extension.js:409
    this._speedButton.connect('clicked', () => this._onSpeedTestClicked())
  • extension.js:145
            this.menu.connect('open-state-changed', (_menu, open) => {
                if (open)
                    this._onMenuOpened();
                else
                    this._onMenuClosed();
            })
  • extension.js:414
    refreshButton.connect('clicked', () => this._refresh(true))
  • extension.js:418
            settingsButton.connect('clicked', () => {
                this._extension.openPreferences();
                this.menu.close();
            })

All Versions

Version Status
1.2.1 (6) Unreviewed
1.2.0 (5) Rejected
1.1.0 (4) Unreviewed
1.0 (3) Active
1.0 (2) Rejected
1.0 (1) Active