Hi,

gentle ping on this.

On Tue, Aug 15 2023 00:11:45 +0900, Lauri Tirkkonen wrote:
> On Mon, Aug 14 2023 10:37:36 +0000, Simon Ser wrote:
> > On Monday, August 14th, 2023 at 12:34, Lauri Tirkkonen <lauri@hacktheplanet.fi> wrote:
> > 
> > > sorry for resurrecting an old thread, but I was wondering: are there still plans
> > > to make this license change?
> > 
> > All copyright holders except Noah have accepted the change. We'd need
> > their ACK, or someone who hasn't looked at the code would need to
> > rewrite their contribution.

On Mon, Aug 14 2023 10:37:36 +0000, Simon Ser wrote:
> On Monday, August 14th, 2023 at 12:34, Lauri Tirkkonen <lauri@hacktheplanet.fi> wrote:
> 
> > sorry for resurrecting an old thread, but I was wondering: are there still plans
> > to make this license change?
> 
> All copyright holders except Noah have accepted the change. We'd need
> their ACK, or someone who hasn't looked at the code would need to
> rewrite their contribution.

Here's my attempt at the latter: from git log,
9df571dc6febd99fe7087cd5d86e3ffbc8051123 looks like the only commit Noah made.
So, I checked out the parent of that commit, and worked from the commit message
and current app behaviour to come up with this diff. It won't apply to master
[message trimmed]

Hi,

On Sun, Jan 15 2023 16:03:27 +0000, Simon Ser wrote:
> Allow developers to distribute the app via the Apple app store.
> 
> References: https://lists.sr.ht/~emersion/goguma-dev/%3CZ_HeiV90GoVvfCDQCre3m3XVfYmLcxEI2GkAq56v3hGszfW1uqpqXSjzoQFlJ-SGuRmuvvewHLO4HN7m9Nq53l2y2tYrbp6azs3LtNFCnpo%3D%40emersion.fr%3E
> ---
> 
> We discussed and decided this was the best course of action.
> 
> delthas, Jean, Noah, Pedro: since you own the copyright, can you reply
> with an ACK?

sorry for resurrecting an old thread, but I was wondering: are there still plans

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

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

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]

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]