#2958 DBConnectionString/dsn option for db connection
Merged by tkopecek. Opened by tkopecek.
tkopecek/koji issue2838  into  master

Download 2958.patch

Alternative DSN connection string fro connecting to postgres. Allow us
to specify e.g. sslmode without adding separate DB* options.

Fixes: https://pagure.io/koji/issue/2838

@julian8628 @ktdreyer @jcupova

:thumbsup:

:thumbsup:

I tested this today and it works great. Thank you!

I pushed a few changes here: https://pagure.io/fork/ktdreyer/koji/commits/db-connection-string . Want to take these into your branch?

I added a walkthrough for administrators to enable SSL using this new feature. I also think we should just remove the old options from the sample configuration file and steer users towards this new setting.

Thanks, added!

Metadata Update from @tkopecek:
- Pull-request tagged with: testing-ready

Metadata Update from @jobrauer:
- Pull-request tagged with: testing-done

Commit dfce69f6 fixes this pull-request

Pull-Request has been merged by tkopecek

Metadata