pagure

#4281 Removed variables from default config as properly defined elsewhere for mqtt
Merged by pingou. Opened by arrfab.
arrfab/pagure mqtt-variables-fix  into  master

Download 4281.patch

We don't want those variables to be in default config, as lib/notify.py will initialize those correctly with python ssl lib if nothing is specified in pagure.cfg :

mqtt_cert_reqs = pagure_config.get("MQTT_CERT_REQS", ssl.CERT_REQUIRED)
mqtt_tls_version = pagure_config.get( "MQTT_TLS_VERSION", ssl.PROTOCOL_TLSv1_2)

Having those two variables in default config set to None breaks mqtt notification (as they don't get correct default values from ssl)

Signed-off-by: Fabian Arrotin fabian.arrotin@arrfab.net

Thanks! :)

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.