Benny Pedersen wrote on 19/01/23 10:47 +13:00:
https://github.com/apache/spamassassin/blob/trunk/lib/Mail/SpamAssassin/Plugin/DecodeShortURLs.pm#L594-L601 only me testing postgresql ?
As I discovered from a recently discovered postgres-only bug, there must not have been much use of postgres with trunk before 4.0.0 was released and not much running of the tests with postgres. I'm in the process of setting up some more automated testing on GitHub actions machines that include the postgres tests.
I'm not an SQL expert. Can you give me more details on how to trigger the bug you are pointing out, what it does, and what is expected?
I don't think that we have any tests in the test suite that cover use of the cache with anything but default SQLite. It would be a good idea to add test coverage for anything that has SQL queries.
Benny Pedersen wrote on 19/01/23 14:04 +13:00:
good i can confirm that only the ttl expire fails, other things works as i can see is github not current trunk code btw ?, i just belive error exists in line number as above
The GitHub mirror updates from svn trunk very quickly after any commit. I haven't timed it, but have never looked at GitHub fast enough to not see a commit I have made.
Sidney
