Skip to content

Fix current LDAP, diagnostics, and workspace feedback - #189

Merged
bifrost0x merged 2 commits into
mainfrom
dev/open-feedback-2026-09-02
Sep 2, 2026
Merged

Fix current LDAP, diagnostics, and workspace feedback#189
bifrost0x merged 2 commits into
mainfrom
dev/open-feedback-2026-09-02

Conversation

@bifrost0x

@bifrost0x bifrost0x commented Sep 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • add an optional TLS-verified LDAP backup endpoint with transport-only failover and safe authentication semantics
  • fix Linux network diagnostics to read byte counters correctly for both short and predictable interface names
  • localize the recent-connection just-now label in every supported locale
  • align Command Sets and Command Library search/action geometry and focus behavior
  • move the File Manager source-search focus ring to the complete compound control
  • keep all six embedded SFTP actions on one row
  • update cache versions, deployment examples, documentation, and regression coverage

Feedback reconciliation

This PR was prepared against current main after checking the previously merged work in #181, #182, and #185. It contains only the still-open follow-ups from:

Verification

  • Python suite: 2320 passed, 2 skipped; the four socket-dependent cases were rerun with local sockets enabled
  • JavaScript unit suite: 35 passed
  • Targeted Playwright suite: 33 passed
  • ESLint for static JavaScript: passed
  • vendored frontend asset check: passed
  • Python compilation and git diff checks: passed

Fixes #187
Fixes #188

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 2, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-02T17:48:02.711166Z 8bf0fe5 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@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: 8bf0fe5ec7

ℹ️ 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 config.py Outdated
@bifrost0x
bifrost0x merged commit ed8e5ee into main Sep 2, 2026
20 of 21 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in WebSSH Roadmap Sep 2, 2026
@bifrost0x
bifrost0x deleted the dev/open-feedback-2026-09-02 branch September 2, 2026 18:12
@bifrost0x bifrost0x self-assigned this Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Network throughput graph in Diagnostics pane Recent Connections card shows time.justNow

1 participant