Thread

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →
  1. Fix missing "use Test::More" in Kerberos.pm.

  1. Small issue with kerberos tests

    Maxim Orlov <orlovmg@gmail.com> — 2025-08-18T17:04:39Z

    Hi!
    
    Recently, I attempted to run the entire set of pg tests with
    PG_TEST_EXTRA="kerberos ..." on NetBSD but encountered
    a problem. The kerberos/001_auth and
    libpq/005_negotiate_encryption.pl test failed with an incomprehensible
    message:
    [01:55:38.737](3.698s) # setting up Kerberos
    # Running: krb5-config --version
    Undefined subroutine &PostgreSQL::Test::Kerberos::BAIL_OUT called at
    /home/omg/proj/postgres/src/test/perl/PostgreSQL/Test/Kerberos.pm line 109.
    
    The root cause of the problem was in using
    ------
    $ /usr/bin/krb5-config --version
    heimdal 7.8.0
    ------
    
    ...insted of..
    
    ------
    $ /usr/pkg/bin/krb5-config --version
    Kerberos 5 release 1.21.3
    ------
    
    In other words, PATH must be properly set to use Kerberos 5 version.
    But, AFIACS, "use Test::More" is missing in Kerberos.pm resulting
    in "Undefined subroutine" error instead of "Heimdal is not supported".
    
    -- 
    Best regards,
    Maxim Orlov.
    
  2. Re: Small issue with kerberos tests

    Tom Lane <tgl@sss.pgh.pa.us> — 2025-08-18T18:41:43Z

    Maxim Orlov <orlovmg@gmail.com> writes:
    > Recently, I attempted to run the entire set of pg tests with
    > PG_TEST_EXTRA="kerberos ..." on NetBSD but encountered
    > a problem. The kerberos/001_auth and
    > libpq/005_negotiate_encryption.pl test failed with an incomprehensible
    > message:
    > [01:55:38.737](3.698s) # setting up Kerberos
    > # Running: krb5-config --version
    > Undefined subroutine &PostgreSQL::Test::Kerberos::BAIL_OUT called at
    > /home/omg/proj/postgres/src/test/perl/PostgreSQL/Test/Kerberos.pm line 109.
    
    Replicated here on FreeBSD (after hot-wiring the script to not know it
    should use /usr/local/bin/krb5-config).
    
    It looks like every other use of BAIL_OUT is in a file that does
    have "use Test::More", so this is the only such oversight.
    Will push, thanks for the report!
    
    			regards, tom lane