Skip to content

Infrastructure#4

Draft
tkogut wants to merge 3 commits intofeature/network-access-enhancementfrom
cursor/infrastructure-b355
Draft

Infrastructure#4
tkogut wants to merge 3 commits intofeature/network-access-enhancementfrom
cursor/infrastructure-b355

Conversation

@tkogut
Copy link
Owner

@tkogut tkogut commented Jan 14, 2026

Improve Mosquitto configuration in Start-Mosquitto-Task.ps1 to align with best practices and enable persistence, logging, and firewall rules.

The previous configuration lacked persistence, logging, and a firewall rule, making the broker less robust and harder to troubleshoot. This PR addresses these issues by adding file-based logging, enabling persistence with dedicated data and log directories, and creating a firewall rule for port 1883, ensuring the Mosquitto instance is more reliable and manageable.


Open in Cursor Open in Web

Task I1: Verified and enhanced Mosquitto configuration in Start-Mosquitto-Task.ps1

Changes:
- Added persistence configuration (true with data directory)
- Added file-based logging (error, warning, notice, information)
- Added Windows Firewall rule creation for port 1883 (Private profile)
- Created data and logs directories with SYSTEM account permissions
- Updated uninstall to also remove firewall rule
- Enhanced summary output with new configuration details

The configuration now has feature parity with Install-Mosquitto.ps1.

Co-authored-by: roostertk <roostertk@tutamail.com>
@cursor
Copy link

cursor bot commented Jan 14, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@vercel
Copy link

vercel bot commented Jan 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
bridge-scale-cmd-rust Ready Ready Preview, Comment Jan 14, 2026 7:57pm

Co-authored-by: roostertk <roostertk@tutamail.com>
I3: Update Inno Setup Installer
- Added 'mqttenable' checkbox for enabling MQTT on startup
- Added Configure-MQTT-Service.ps1 to installer files
- Added Run entry to configure MQTT when checkbox selected

I4: Add MQTT to CI/CD pipeline
- Added MQTT environment variables to global env section
- Added Mosquitto installation for E2E tests
- Configured MQTT_ENABLED for test environments

I5: Create Configure-MQTT-Service.ps1 script
- Full PowerShell script for MQTT configuration
- Supports -Enable, -Disable, -Status operations
- Configurable broker URL, client ID, topic prefix, auth, QoS
- Uses NSSM for service environment management

I6: Update Windows Installation Documentation
- Added MQTT Configuration section
- Added Mosquitto Installation section
- Added MQTT Troubleshooting section
- Updated table of contents

Co-authored-by: roostertk <roostertk@tutamail.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.

2 participants