Skip to content

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

Open
64johnlee wants to merge 21 commits into
RetroShare:masterfrom
64johnlee:ralph/RS-088
Open

feat(config): display Tor hidden address and status#109
64johnlee wants to merge 21 commits 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

- Add displayHiddenStatus component showing hidden mode (Enabled/Disabled)
- Add displayHiddenAddress component showing hiddenNodeAddress and hiddenPort
- Both components use details from /rsPeers/getPeerDetails API
- Integrated into Network Configuration grid in config_network.js
- Also updated config_node.js Public Information to show hidden address/port

Fixes issue RetroShare#88 - Tor Status not works and hidden address display in settings is missed
@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.

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