Skip to content

Initial TLS support#30

Open
Adminiuga wants to merge 2 commits intommcc-xx:masterfrom
Adminiuga:ac/mqtt_tls
Open

Initial TLS support#30
Adminiuga wants to merge 2 commits intommcc-xx:masterfrom
Adminiuga:ac/mqtt_tls

Conversation

@Adminiuga
Copy link
Copy Markdown

Use TLS for broker connection.
The following configuration file parameters were added:

Parameter Required Default Value Notes
mqtt_port False 1883 MQTT Broker connection port
mqtt_use_tls False False Use TLS for broker connection
mqtt_insecure_tls False False Don't verify broker certificate
mqtt_tls_ca_certs False None Path to .pem file containing all trusted root certificates

k1n6b0b added a commit to k1n6b0b/WhosAtMyFeeder that referenced this pull request Mar 9, 2026
Cherry-picked from upstream PR mmcc-xx#30 by @Adminiuga
(mmcc-xx#30)

Adds optional MQTT configuration:
- mqtt_port: custom broker port (default: 1883)
- mqtt_use_tls: enable TLS encryption (default: false)
- mqtt_tls_insecure: skip certificate verification (default: false)
- mqtt_tls_ca_certs: path to CA certificate file (optional)

Fix: corrected DEFAULT_MQTT_PORT typo from 1833 → 1883 in original PR.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
k1n6b0b added a commit to k1n6b0b/WhosAtMyFeeder that referenced this pull request Mar 9, 2026
Cherry-picked from upstream PR mmcc-xx#30 by @Adminiuga
(mmcc-xx#30)

Adds optional MQTT configuration:
- mqtt_port: custom broker port (default: 1883)
- mqtt_use_tls: enable TLS encryption (default: false)
- mqtt_tls_insecure: skip certificate verification (default: false)
- mqtt_tls_ca_certs: path to CA certificate file (optional)

Fix: corrected DEFAULT_MQTT_PORT typo from 1833 → 1883 in original PR.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
k1n6b0b added a commit to k1n6b0b/WhosAtMyFeeder that referenced this pull request Mar 9, 2026
Cherry-picked from upstream PR mmcc-xx#30 by @Adminiuga
(mmcc-xx#30)

Adds optional MQTT configuration:
- mqtt_port: custom broker port (default: 1883)
- mqtt_use_tls: enable TLS encryption (default: false)
- mqtt_tls_insecure: skip certificate verification (default: false)
- mqtt_tls_ca_certs: path to CA certificate file (optional)

Fix: corrected DEFAULT_MQTT_PORT typo from 1833 → 1883 in original PR.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
k1n6b0b added a commit to k1n6b0b/WhosAtMyFeeder that referenced this pull request Mar 9, 2026
Cherry-picked from upstream PR mmcc-xx#30 by @Adminiuga
(mmcc-xx#30)

Adds optional MQTT configuration:
- mqtt_port: custom broker port (default: 1883)
- mqtt_use_tls: enable TLS encryption (default: false)
- mqtt_tls_insecure: skip certificate verification (default: false)
- mqtt_tls_ca_certs: path to CA certificate file (optional)

Fix: corrected DEFAULT_MQTT_PORT typo from 1833 → 1883 in original PR.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
k1n6b0b added a commit to k1n6b0b/WhosAtMyFeeder that referenced this pull request Mar 9, 2026
- Fix threshold in config.yml.example: 0.55 → 0.7 (matches upstream default)
- Add MQTT TLS config options to example (from merged feat/mqtt-tls)
- Rewrite README: cleaner structure, accurate feature list, calls out
  cherry-picked PRs (mmcc-xx#30 MQTT TLS, mmcc-xx#46 MQTT publish) with attribution,
  removes stale upstream "What's New" notes, adds Development section

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant