This will properly close the database connection after a Postgres WAL backup was made. As far as I could see a call to close() is completely missing in the current code. As the the connection may already been closed I put it in a try/except to ignore all errors that may occur.

On my test system this prevents the PostgreSQL server to log

unexpected EOF on client connection with an open transaction

every backup.

• Short description and the purpose of this PR is present
• Your name is present in the AUTHORS file (optional)

If you have any questions or problems, please give a comment in the PR.

Helpful documentation and best practices

• Git Workflow
• Automatic Sourcecode Formatting
• Check your commit messages

Checklist for the reviewer of the PR (will be processed by the Bareos team)

General

• PR name is meaningful
• Purpose of the PR is understood
• Commit descriptions are understandable and well formatted
• Check backport line
• Is the PR title usable as CHANGELOG entry?
• Separate commit for CHANGELOG.md ("update CHANGELOG.md"). The PR number is correct.

Source code quality

• Source code changes are understandable
• Variable and function names are meaningful
• Code comments are correct (logically and spelling)
• Required documentation changes are present and part of the PR
• bareos-check-sources --since-merge does not report any problems