Message-ID: <gh-pgjdbc-pgjdbc-2112-c2095000109@github.com>
From: "stuartwdouglas (@stuartwdouglas)" <noreply+stuartwdouglas@github.com>
To: "pgjdbc/pgjdbc" <noreply+pgjdbc-pgjdbc@github.com>
Date: Sun, 05 May 2024 23:42:41 +0000
Subject: Re: [pgjdbc/pgjdbc] issue #2112: Single argument version of Class.forName is used to load classes, which can cause problems with non-flat class paths
In-Reply-To: <gh-pgjdbc-pgjdbc-2112@github.com>
References: <gh-pgjdbc-pgjdbc-2112@github.com>
List-Id: <gh-pgjdbc-pgjdbc.github.com>
X-GitHub-Author-Login: stuartwdouglas
X-GitHub-Comment-Id: 2095000109
X-GitHub-Comment-Type: issue_comment
X-GitHub-Issue: 2112
X-GitHub-Repo: pgjdbc/pgjdbc
X-GitHub-Type: comment
X-GitHub-Url: https://github.com/pgjdbc/pgjdbc/issues/2112#issuecomment-2095000109
Content-Type: text/plain; charset=utf-8

Re-visiting this would you be open to a fix that allows this behaviour to be controlled by a config property of some kind (e.g. a system property, or some other form of config)?

Personally I don't believe that my PR will break anything, as any environment that has set the TCCL would be expecting it to be used, and the fallback means it will continue to work in environments where it is not set, but if this is a concern then I think a config option is the only way to fix this.