Updated Now - #1696
Conversation
|
Done here |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 83944c188c
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| relay.agilesolutionlabs.com,43.6532,-79.3832 | ||
| nostr.sovereignservices.xyz,43.6532,-79.3832 | ||
| relay.qstr.app,50.1109,8.68213 | ||
| articles.layer3.news:443,37.3387,-121.885 |
There was a problem hiding this comment.
Remove duplicate aliases for default port 443
For users selecting a geohash near San Jose, this entry and articles.layer3.news at line 108 both rank among the five closest relays. GeoRelayDirectory.parseCSV treats their host strings as distinct, even though wss://articles.layer3.news and wss://articles.layer3.news:443 reach the same default TLS endpoint, so the fixed-size result opens two connections to one server and provides only four distinct relays. This update introduces 78 such alias pairs; normalize the default port or remove the duplicates to preserve relay redundancy.
Useful? React with 👍 / 👎.
Updated the project after serveral commit