Bug 261514

Summary: www/onlyoffice-documentserver add newsyslog script
Product: Ports & Packages Reporter: Martin Waschbüsch <martin>
Component: Individual Port(s)Assignee: Mikael Urankar <mikael>
Status: Closed FIXED    
Severity: Affects Only Me Flags: bugzilla: maintainer-feedback? (mikael)
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
suggested newsyslog config file
none
suggested newsyslog config file none

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!