Recent activity

Re: [PATCH gamja] add TLS checkbox to network form 4 months ago

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

Re: [PATCH] server: do not require restart for newly added users 4 months ago

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 :)

[PATCH gamja] add TLS checkbox to network form 4 months ago

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]

[PATCH] server: do not require restart for newly added users 4 months ago

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]