Hey, Landry. Thanks for looking into it.

> it doesnt make sense to run as www user. ideally it should run as its
> own separate user.
Indeed that is ideal, however acquiring new UIDs is a sensitive matter
so I fallback to using 'www' for initial reviewing/testing purposes just
in case.

I've updated the port to run the daemon as a separate user. This is now
actually how I use it on my machine. Don't forget to amend the user's
list. Diff at the bottom. Hopefully my e-mail client hasn't mangled it.

> README mentioning rc flags -> those should be the defaults in the rc
> script itself (eg daemon_flags)
> when running as a rc daemon, wouldnt it make snse to enable --sessions
> ? thinking about multiple users connecting to pgweb, and wanting to
> connect to distinct databases.

Sure. We can do it that way if that's how it's generally preferable. The
new port version now has default flags plus the 'session' option set in
RC. The readme is also updated to reflect this change.

Attached is the version with all the above suggestions applied. Feedback
appreciated.

On Wed, 2026-03-04 at 09:11 +0100, Landry Breuil wrote:
> Le Sat, Feb 28, 2026 at 07:39:42PM +0100, Igor Zornik a écrit :
> > On Sat, 2026-02-21 at 11:07 +0100, Igor Zornik wrote:
> > > Hello, ports!
> > > 
> > > Attached is a submission for a new port. Pgweb is web-based client
> > > for
> > > working with PostgreSQL databases. I like this one because it is
> > > simple
> > > to deploy and has enough features to be generally useful. You can
> > > run
> > > it
> > > standalone or as a daemon through RC. In the later case I’ve set
> > > up
> > > the port to run it as user “www” for now.
> > > 
> > > Any feedback is welcome. Package info output:
> 
> i've tested it locally, and it works fine. For the daemon mode, there
> are still improvements/things to fix imo:
> 
> - it doesnt make sense to run as www user. ideally it should run as
> its
>   own separate user.
> - README mentioning rc flags -> those should be the defaults in the rc
>   script itself (eg daemon_flags)
> - when running as a rc daemon, wouldnt it make snse to enable --
> sessions ?
>   thinking about multiple users connecting to pgweb, and wanting to
> connect to distinct databases..
> 
> Landry
> 

Index: user.list
===================================================================
RCS file: /cvs/ports/infrastructure/db/user.list,v
retrieving revision 1.482
diff -u -p -u -p -r1.482 user.list
--- user.list   16 Feb 2026 13:57:37 -0000      1.482
+++ user.list   4 Mar 2026 21:01:52 -0000
@@ -412,3 +412,4 @@ id  user            group           port
 901 _ergo              _ergo           net/ergo
 902 _hylafax           _hylafax        comms/hylafax
 903 _iaxmodem          _iaxmodem       telephony/iaxmodem
+904 _pgweb             _pgweb  www/pgweb

Attachment: pgweb.tgz
Description: application/compressed-tar

Reply via email to