error_log, access_log tackle syslog (#185)

[Retssaze]

Proposed changes

Describe the use case and detail of the change. If this PR addresses an issue on GitHub, make sure to include a link to that issue using one of the supported keywords here in this description (not in the title of the PR).

Checklist

Before creating a PR, run through this checklist and mark each as complete.

• I have read the CONTRIBUTING document
• I have run make install-tools and have attached any dependency changes to this pull request
• If applicable, I have added tests that prove my fix is effective or that my feature works
• If applicable, I have checked that any relevant tests pass after adding my changes
• If applicable, I have updated any relevant documentation (README.md)
• If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

[netlify]

✅ Deploy Preview for agent-public-docs ready!

Name	Link
🔨 Latest commit	25a2bd6
🔍 Latest deploy log	https://app.netlify.com/sites/agent-public-docs/deploys/64832ad81302d50008630c88
😎 Deploy Preview	https://deploy-preview-302--agent-public-docs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[oliveromahony]

Thanks for the contribution @Retssaze - can you run a make deps and commit the changes there? Also if you could add a unit test for these scenarios, it would be fantastic

[oliveromahony]

Also lint seems to be failing

[Retssaze]

Dear @olivermahony, thank you very much for your mail

make deps does not work

[Dean-Coakley]

@Retssaze I think for make deps to work, you need to run make install-tools first. See https://github.com/nginx/agent#development-environment-setup for more details.

[Dean-Coakley]

@Retssaze Did you get a chance to try running make install-tools before running make deps? - If it's still broken can you share the output of the command?

If you can run make deps successfully and resolve the minor lint issues this should be good to merge.

[Retssaze]

Dear @Dean-Coakley, thanks for the comment. Would do it during the upcoming week пятница, 26 мая 2023 г. пользователь Dean Coakley ***@***.***> написал:

@Retssaze Did you get a chance to try running make install-tools before

running make deps? - If it's still broken can you share the output of the command?

If you can run make deps successfully and resolve the minor lint issues

this should be good to merge.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.<

https://ci3.googleusercontent.com/proxy/hyrIAgZgS25lZ9bgiRhFwGl7PZ-Sy80VRycygR06fRF-R8Xr4KGWxgdSM75qxHSk7YuyWmB8SVuR_iilIWT1jhvmvYKVqA77eixXZSlI5qygpBAzks6pQfv9mI141BC-nTIdIi71yZS7HiDetKkM63fM1ofVkU67H8VNqLVAnBSqfsJsyWC0plwqjzypDbDuxxYKaEp9JlhF2TAv9lOI5ty5Tyu8JJzdWdrpvy29CRr0gvaedQ=s0-d-e1-ft#https://github.com/notifications/beacon/AS4YXNXCZNPS6VUQUYUIJWDXIDEIJA5CNFSM6AAAAAAXI2JZTKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTS5IFTOA.gif>Message ID: ***@***.***>

[oliveromahony]

Updated outstanding items - ready to merge now