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.

Sign up for GitHub

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

Jump to bottom

Remove ansi codes from syslog when running in background #170

Closed
rrosson opened this issue Apr 13, 2020 · 3 comments
Closed

Remove ansi codes from syslog when running in background #170

rrosson opened this issue Apr 13, 2020 · 3 comments
Labels
enhancement released

Comments

@rrosson
Copy link

rrosson commented Apr 13, 2020

Is your feature request related to a problem? Please describe.
Remove ansi codes from syslog (i.e. when room-assistant is started from systemctl)

Describe the solution you'd like
Remove ansi codes from syslog

Describe alternatives you've considered
Will make it easier to check status of room-assistant operation

Additional context
Add any other context or screenshots about the feature request here.
@mKeRix
Copy link
Owner

mKeRix commented Apr 13, 2020

Can you provide an example of which codes/characters you mean in the logs? The logs look fine in systemctl status/journald for me.

@rrosson
Copy link
Author

rrosson commented Apr 13, 2020 

# grep room /var/log/syslog
Apr 13 08:26:43 ra-roffice room-assistant[25368]: #033[32m[Nest] 25368   - #033[39m04/13/2020, 8:26:43 AM   #033[38;5;3m[ConfigService] #033[39m#033[32mLoading configuration from /opt/nodejs/lib/node_modules/room-assistant/dist/config/definitions/default.js, config/local.yml#033[39m
Apr 13 08:26:44 ra-roffice room-assistant[25368]: #033[32m[Nest] 25368   - #033[39m04/13/2020, 8:26:44 AM   #033[38;5;3m[NestApplication] #033[39m#033[32mNest application successfully started#033[39m#033[38;5;3m +169ms#033[39m
Apr 13 08:27:03 ra-roffice room-assistant[25368]: #033[32m[Nest] 25368   - #033[39m04/13/2020, 8:27:03 AM   #033[38;5;3m[ClusterService] #033[39m#033[32mAdded 192.168.65.211:6425 to the cluster with id ra-shop#033[39m
```

@mKeRix mKeRix closed this as completed in 0f90eaa Apr 13, 2020
github-actions bot pushed a commit that referenced this issue Apr 13, 2020
@semantic-release-bot
chore(release): 2.4.0 [skip ci]
5da8be0 
# [2.4.0](v2.3.0...v2.4.0) (2020-04-13)

### Bug Fixes

* include distances in API for room presence sensors ([567327d](567327d))
* use Tini for process management in Docker ([932d603](932d603)), closes [#157](#157)

### Features

* simplify log format ([0f90eaa](0f90eaa)), closes [#170](#170)
* **bluetooth-classic:** allow device-specific minRssi values ([cf3ddc5](cf3ddc5)), closes [#168](#168)
* **bluetooth-low-energy:** allow the update frequency to be throttled ([0143309](0143309)), closes [#125](#125)
@github-actions
Copy link

🎉 This issue has been resolved in version 2.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement released
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rrosson @mKeRix