Skip to content

feat: adding the mqtt-antena app#5960

Open
fbossolan wants to merge 3 commits into
bigbeartechworld:masterfrom
fbossolan:feat/mqtt-antena
Open

feat: adding the mqtt-antena app#5960
fbossolan wants to merge 3 commits into
bigbeartechworld:masterfrom
fbossolan:feat/mqtt-antena

Conversation

@fbossolan

Copy link
Copy Markdown

This repo adds the files for the MQTT Antena application. The main documentation of the project can be found here.

Tests:

The app was successfully built as a container using this configuration in a test CasaOS installation:

Installed app:
test-installation-2


Inside the app:
test-installation1

@coderabbitai

coderabbitai Bot commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • New Features
    • Added MQTT Antena app with configuration and deployment so the service is now installable and runnable.
    • Service is exposed on port 8585 for access.
    • Includes app metadata and a docs link for user guidance.

Walkthrough

Adds a new MQTT Antena application: a static metadata file and a docker-compose service definition (image, port, volume, restart) plus CASAOS metadata for the service.

Changes

Cohort / File(s) Summary
MQTT Antena app files
Apps/MQTT-Antena/config.json, Apps/MQTT-Antena/docker-compose.yml
Adds app metadata JSON (id, version, image ref, docs/YouTube placeholders) and a docker-compose service mqtt-antena using image fbossolan/mqtt-antena:1.5.8, exposing port 8585, mounting a local data volume, restart: unless-stopped, and including x-casaos metadata (architectures amd64/arm64, category IOT, author/developer, localized title/description, icon, port_map).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 I stitched a tiny config bright,
A compose to make it hum at night,
Ports and volumes snug and neat,
MQTT hops on tiny feet,
Hop, deploy — the antenna's right! 📡✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'feat: adding the mqtt-antena app' clearly summarizes the main change - adding a new MQTT Antena application with configuration files.
Description check ✅ Passed The description is directly related to the changeset, explaining that it adds files for the MQTT Antena application with documentation links and testing evidence.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
Apps/MQTT-Antena/docker-compose.yml (2)

22-22: Icon URL: prefer raw.githubusercontent.com and remove trailing whitespace.

The current URL goes through a GitHub redirect (302) on every icon load. Using the direct raw URL avoids the redirect and is more resilient to GitHub UI changes. Also, there's a trailing space on this line.

Proposed fix
-  icon: https://github.com/fbossolan/mqtt-antena/blob/main/src/static/img/logo.png?raw=true 
+  icon: https://raw.githubusercontent.com/fbossolan/mqtt-antena/main/src/static/img/logo.png

1-2: Nit: remove the leading blank line.

The file starts with an empty line before the name: directive. Removing it keeps the file consistent with typical docker-compose conventions.

Proposed fix
-
 name: big-bear-mqtt-antena

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@Apps/MQTT-Antena/docker-compose.yml`:
- Around line 11-12: Replace the hardcoded placeholder SECRET_KEY in
docker-compose.yml with a secure default behavior: remove the static "-
SECRET_KEY=your_secret_key_here" and instead make the container/entrypoint or
application generate a cryptographically secure random key when the environment
variable SECRET_KEY is unset; update the entrypoint (or app startup logic) to
check for process.env.SECRET_KEY (or ENV SECRET_KEY) and create/persist a random
secret if absent, and change the compose file to either omit SECRET_KEY entirely
or set a clear placeholder like "REQUIRED_CHANGE" to force explicit
configuration.
🧹 Nitpick comments (2)
Apps/MQTT-Antena/docker-compose.yml (2)

1-1: Leading blank line.

Line 1 is empty before the name: directive. Minor nit — other compose files in the repo likely start on line 1 without a leading blank.


24-24: Trailing whitespace on the icon URL line.

There's a trailing space after the icon URL on line 24. While harmless, it's best to trim it for cleanliness.

Comment thread Apps/MQTT-Antena/docker-compose.yml Outdated
@dragonfire1119

Copy link
Copy Markdown
Contributor

Thank you for your contribution and for taking the time to add MQTT Antena to the app store! I really appreciate the effort you've put into this.

However, we're no longer accepting pull requests to the big-bear-casaos repository. We've transitioned to a universal repository approach to better support multiple platforms.

Please submit your PR to our new universal repository instead:
https://github.com/bigbeartechworld/big-bear-universal-apps

This new repository consolidates our app store across CasaOS, ZimaOS, and other platforms, making it easier to maintain and ensuring broader compatibility.

Thank you for your understanding, and I look forward to seeing your contribution in the universal repo!

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