Skip to content
This repository has been archived by the owner on Aug 25, 2021. It is now read-only.

Draft: Implements the logs sender for Jason #74

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Conversation

Kirguir
Copy link
Contributor

@Kirguir Kirguir commented Dec 10, 2019

Synopsis

Need implements send errors, debug info or any other string info to media server.

Solution

Save the information that you want to send to the server in the local storage of the browser.
Run Log Sender in the background, which periodically checks the local storage and, if there is information to send, sends it a POST request to the server.

Checklist

  • Created PR:
    • In draft mode
    • Name contains Draft: prefix
    • Name contains issue reference
    • Has k:: labels applied
    • Has assignee
  • Documentation is updated (if required)
  • Tests are updated (if required)
  • Changes conform code style
  • CHANGELOG entry is added (if required)
  • FCM (final commit message) is posted
    • and approved
  • Review is completed and changes are approved
  • Before merge:
    • Milestone is set
    • PR's name and description are correct and up-to-date
    • Draft: prefix is removed
    • All temporary labels are removed

- add proxy for POST /logs in nginx config for demo
- send Room errors to server
- move `push_to_store()` to impl LogSender
- impl macro `store_jason_error!`
@Kirguir Kirguir added feature New feature or request k::logging Related to application logs labels Dec 10, 2019
@Kirguir Kirguir assigned Kirguir and alexlapa and unassigned Kirguir Dec 10, 2019
@tyranron tyranron changed the title WIP: Implements the logs sender for Jason Draft: Implements the logs sender for Jason Aug 24, 2020
@tyranron tyranron force-pushed the master branch 5 times, most recently from 10e5ade to 39716ed Compare February 1, 2021 15:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature New feature or request k::logging Related to application logs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants