Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added status file. removed blank line in ircddblocal.php #134

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ea3ihi
Copy link

@ea3ihi ea3ihi commented Apr 18, 2021

Added a status script (in api folder) to expose the configuration and status of the pi-star as json. The idea is to be able to monitor the status of several pi-stars from an iot server.

I have removed extra lines at the end of ircddblocal.php to prevent those lines from being sent with the script output

@MW0MWZ
Copy link
Collaborator

MW0MWZ commented Apr 24, 2021

Its a nice idea, but exposes passwords - so I can't add it in its current state.
I'll take a look at this later to make sure it strips password fields.

@ea3ihi
Copy link
Author

ea3ihi commented Apr 24, 2021

Yes, exposing the full config was too much, I have a new version that does not expose the config but a limited subsed, will try to create a PR from the updated version

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