Skip to content

feat(config): display Tor hidden address and status#109

Open
64johnlee wants to merge 1 commit into
RetroShare:masterfrom
64johnlee:ralph/RS-088
Open

feat(config): display Tor hidden address and status#109
64johnlee wants to merge 1 commit into
RetroShare:masterfrom
64johnlee:ralph/RS-088

Conversation

@64johnlee
Copy link
Copy Markdown

Summary

Fix issue #88 - Tor Status not works and hidden address display in settings is missed.

Changes

  • config_network.js: Added displayHiddenStatus component showing hidden mode (Enabled/Disabled)
  • config_network.js: Added displayHiddenAddress component showing hiddenNodeAddress and hiddenPort
  • config_node.js: Updated Public Information to also display hidden address/port when enabled
  • Uses existing /rsPeers/getPeerDetails API which provides isHiddenNode, hiddenNodeAddress, hiddenPort fields

Files changed

  • webui-src/app/config/config_network.js
  • webui-src/app/config/config_node.js

Testing

  1. Go to Config > Network
  2. Verify "Hidden Mode" and "Hidden Address" sections appear in Network Configuration
  3. When Tor hidden service is enabled, hidden address/port will display

@defnax
Copy link
Copy Markdown
Contributor

defnax commented May 14, 2026

but this is not clean,

  • 3 file changes but 25 commits
  • and not needed file
image

@64johnlee
Copy link
Copy Markdown
Author

Hi @defnax, I've cleaned up the branch. The 24 chore: bump PR commits and any unwanted files have been removed. The branch now has a single clean feature commit:

feat(config): display Tor hidden address and status

Ready for review.

@64johnlee
Copy link
Copy Markdown
Author

Thanks for the feedback. Cleaned up the branch — squashed the bump commits down to the single meaningful commit and removed the stray AGENTS.md file.

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.

2 participants