-
-
Notifications
You must be signed in to change notification settings - Fork 62
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
add notify log #65
Comments
Any plans to add persistent profiles and send records? |
Sorry for the delays. It's not that I'm ignoring you. I just haven't had time to have a further look into this. Hopefully on the weekend that will change.
Persistent storage already exists. But I guess you mean enabling Django Auth and hide them behind accounts? As per the logs, this is doable but I would think Python logging to disk might not be as efficient as maybe just letting nginx write to the same persistent storage location as the / |
Thank you very much for your reply, I'm very interested in this project, I've only read some code so far because I guess what we are missing is a dashboard to view or manage apprise call logs, nginx logs do do that but obviously not easy to manage. On the other hand, apprise calls can fail and we need these logs to troubleshoot problems. Adding a retry mechanism to include a queue seems to run counter to the lightweight design. Maybe I need to reconsider it .... |
No need to reconsider; I love feedback. I just need to think about it a bit more. The problem is that there already is logging in place but it's at the web hosting level. I get that you'd like to gather more usage statistics i think...
This is the kind of useful stuff that would go into a database or some key/value store (nosql) solution. With a backend, then you can also add user authentication, etc. Quiet honestly even a message broker would be useful so that the notifications are sent/handled after your POST request (not during). I'll ponder just the logging idea; but with the current design; i can't think of a simple way to do it (without starting the steps identified above). Consider that there could be concurrent hits for notification to be sent as well. So just writing to a view-able log file (retrievable through a Django view) would involve semaphore/mutex (locking) etc which may not be the most ideal approach. Those are just my thoughts out loud though... 🙂 |
Thanks a lot @caronc , it's really not necessary to integrate too much in |
This request aligns with #143 |
💡 The Idea
add a simple dashboard to show notify log
🔨 Breaking Feature
maybe django admin is a good idea
The text was updated successfully, but these errors were encountered: