From Lauri Tirkkonen to ~emersion/public-inbox
Thanks for your response. On Mon, Jan 16 2023 10:52:52 +0000, Simon Ser wrote: > Hm, thanks for the patch, but I'm not sure this is something I want to > support. I'd prefer to force users who want insecure connections to use > BouncerServ manually. Insecure connections are best avoided. Sure. It's just that there are some IRC networks out there that either don't support TLS connections at all (QuakeNet, I think), or only a small minority of servers do (IRCnet); I wanted my users to have an easier experience with those. That said, the patch does change behavior for irc:// links - I could be persuaded into requiring irc+insecure:// for insecure links instead, if
From Lauri Tirkkonen to ~emersion/soju-dev
On Mon, Jan 16 2023 10:51:33 +0000, Simon Ser wrote: > Thanks for the patch, but this feature has already been rejected in the > past: > > https://lists.sr.ht/~emersion/soju-dev/patches/25249 > https://lists.sr.ht/~emersion/soju-dev/patches/26042 Ah, I see, thanks for the explanation, and sorry for the noise. I will maintain a local patch for this then. Out of curiosity, is the current behavior not problematic for chat.sr.ht? I would imagine larger deployments would generally not want to restart the bouncer :)
From Lauri Tirkkonen to ~emersion/public-inbox
also, remove the default value for the port field; per the bouncer-networks spec, if it's missing the bouncer should automatically choose the appropriate port (6667 or 6697, depending on tls) --- components/app.js | 4 ++-- components/network-form.js | 10 +++++++++- lib/irc.js | 16 +++++++++++++--- 3 files changed, 24 insertions(+), 6 deletions(-) diff --git a/components/app.js b/components/app.js index 48dc72b..53bb946 100644 --- a/components/app.js +++ b/components/app.js @@ -1304,6 +1304,7 @@ export default class App extends Component { [message trimmed]
From Lauri Tirkkonen to ~emersion/soju-dev
if the user exists in the database but was not loaded on daemon startup, attempt to load it from the database before instructing the operator to restart the daemon. --- server.go | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/server.go b/server.go index 7d1d305..79c19f2 100644 --- a/server.go +++ b/server.go @@ -390,6 +390,13 @@ func (s *Server) forEachUser(f func(*user)) { func (s *Server) getUser(name string) *user { s.lock.Lock() [message trimmed]