Bug 261514 - www/onlyoffice-documentserver add newsyslog script
Summary: www/onlyoffice-documentserver add newsyslog script
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Mikael Urankar
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-01-27 11:08 UTC by Martin Waschbüsch
Modified: 2022-02-18 14:17 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (mikael)


Attachments
suggested newsyslog config file (796 bytes, text/plain)
2022-01-27 11:08 UTC, Martin Waschbüsch
no flags Details
suggested newsyslog config file (454 bytes, text/plain)
2022-01-27 12:32 UTC, Martin Waschbüsch
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Waschbüsch 2022-01-27 11:08:31 UTC
Created attachment 231384 [details]
suggested newsyslog config file

Hi there,

a newsyslog config file may be a good addition.
I attached a (for me) working example.
I do not know if this should be installed by default or placed with example files (like the nginx config), though?
I stuck to the permissions and ownership of the log files as created by the port. Though I do not understand why they are set the way they are? (pwner root:onlyoffice with 644 perms).
Comment 1 Martin Waschbüsch 2022-01-27 12:32:39 UTC
Created attachment 231389 [details]
suggested newsyslog config file

originally attached the verbose version I used to try stuff.
This is the concise one.
Comment 2 Martin Waschbüsch 2022-01-27 12:34:19 UTC
Comment on attachment 231389 [details]
suggested newsyslog config file

># logfilename						[owner:group]	mode	count	size	when	flags	[/cmd_file]
>/var/log/onlyoffice/documentserver/{converter,docservice,metrics}/*.log	root:onlyoffice	644	7	*	@T00	BGJ	/var/run/supervisor/supervisord.pid
Comment 3 commit-hook freebsd_committer freebsd_triage 2022-02-18 14:15:46 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=d8171126acd5a857c056959218c8401743971444

commit d8171126acd5a857c056959218c8401743971444
Author:     Mikael Urankar <mikael@FreeBSD.org>
AuthorDate: 2022-02-18 14:13:57 +0000
Commit:     Mikael Urankar <mikael@FreeBSD.org>
CommitDate: 2022-02-18 14:13:57 +0000

    www/onlyoffice-documentserver: Improve port.

      - Option'ify the database backend and add MySQL
      - Add a newsyslog sample file (pr 261514, submitted by Martin Waschbüsch)
      - Add a missing nginx sample file (ds-ssl.conf)
      - Fix an hardcoded /usr/local in files/patch-document-server-package_Makefile
      - Improve pkg-message:
        - Add an 'upgrade' note (suggested by bapt@)
        - Add instructions to use MySQL as a database server
        - Pass the erlang-cookie option when invoking rabbitmqctl

    PR:             261514

 www/onlyoffice-documentserver/Makefile             | 22 +++++++++---
 .../files/onlyoffice.newsyslog.sample.in (new)     |  3 ++
 .../files/patch-document-server-package_Makefile   |  6 ++--
 www/onlyoffice-documentserver/files/pkg-message.in | 42 ++++++++++++++--------
 www/onlyoffice-documentserver/pkg-plist            |  2 ++
 5 files changed, 54 insertions(+), 21 deletions(-)
Comment 4 Mikael Urankar freebsd_committer freebsd_triage 2022-02-18 14:17:51 UTC
Thanks!