MIME-Version: 1.0
References: <33b7c479-c85d-0d48-4971-1b093925e326@pgug.de>
In-Reply-To: <33b7c479-c85d-0d48-4971-1b093925e326@pgug.de>
From: Magnus Hagander <magnus@hagander.net>
Date: Tue, 25 Jan 2022 18:03:59 +0100
Message-ID: 
 <CABUevEw4OnMDqXeuhc4N0WYmTjjibBRP9bQJGX4==OaNPRm4Hg@mail.gmail.com>
Subject: Re: Update Commitfest requirements and README
To: "Andreas 'ads' Scherbaum" <ads@pgug.de>
Cc: pgsql-www@lists.postgresql.org
Content-Type: text/plain; charset="UTF-8"
Archived-At: 
 <https://www.postgresql.org/message-id/CABUevEw4OnMDqXeuhc4N0WYmTjjibBRP9bQJGX4%3D%3DOaNPRm4Hg%40mail.gmail.com>
Precedence: bulk

On Sat, Jan 22, 2022 at 6:34 PM Andreas 'ads' Scherbaum <ads@pgug.de> wrote:
>
>
> Hello,
>
> While trying something out on the Commitfest, I found that the README
> and the dependencies are quite outdated. It still requires Django 1.8, as example.
> And a couple modules are missing, which are not listed in the dependencies.
>
> Attached patch takes a first shot at this problem, updates dependencies and
> README to Django 3.2 and adds all missing modules (so far).

Yeah, this looks cvorrect. Applied.


> This brings the website into a state where it at least starts.
> But then when I try to login, I run into this error:
>
> 'Settings' object has no attribute 'PGAUTH_REDIRECT'
>
> Apparently the example settings for local_settings.py are incomplete.

yeah. You need to set PGAUTH_REDIRECT and PGAUTH_KEY to have it
integrate with a local pgweb. I think the intention was at some point
to make it usable without that, and just with local accounts, but it
doesn't appear to be -- either that was never done or it has gone
broken over time. I think you can make it work locally by just
commenting out all references to the "pgcommitfest.auth" module from
urls.py?

-- 
 Magnus Hagander
 Me: https://www.hagander.net/
 Work: https://www.redpill-linpro.com/