You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by "moying2026 (via GitHub)" <gi...@apache.org> on 2026/05/19 11:52:10 UTC
[PR] fix: correct loggly syslog emergency severity [apisix]
moying2026 opened a new pull request, #13390:
URL: https://github.com/apache/apisix/pull/13390
### Description
This PR corrects the syslog emergency severity name used by the `loggly` plugin and the RFC5424 utility from `EMEGR` to `EMERG`.
`EMERG` is the standard syslog severity name for severity value `0`. The current typo makes `severity: "EMERG"` fail schema validation and the docs currently advertise the misspelled value.
### Checklist
- [x] Correct `loggly` severity enum from `EMEGR` to `EMERG`
- [x] Correct `apisix.utils.rfc5424` severity map from `EMEGR` to `EMERG`
- [x] Update English and Chinese loggly plugin docs
- [x] Add a regression case to ensure `EMERG` is accepted and the old misspelling is rejected
### Tests
- [x] `git diff --check`
- [ ] `make test TEST_FILE=t/plugin/loggly.t` (not run successfully in my local environment because OpenResty is not installed: `WARNING: OpenResty not found`)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: correct loggly syslog emergency severity [apisix]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #13390:
URL: https://github.com/apache/apisix/pull/13390#issuecomment-5252040242
This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the dev@apisix.apache.org list. Thank you for your contributions.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: correct loggly syslog emergency severity [apisix]
Posted by "moying2026 (via GitHub)" <gi...@apache.org>.
moying2026 commented on PR #13390:
URL: https://github.com/apache/apisix/pull/13390#issuecomment-4494132133
补充一下背景：这个 PR 修正了 loggly 插件和 rfc5424 工具里 syslog emergency 级别的拼写错误（EMEGR → EMERG），同时更新了中英文文档，并补了一个回归测试。本地因为没装 OpenResty，没能跑完整套测试，但改动范围很小，只涉及字符串修正和文档，不影响其他逻辑。麻烦有空时帮忙 review，谢谢。
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: correct loggly syslog emergency severity [apisix]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #13390:
URL: https://github.com/apache/apisix/pull/13390#issuecomment-5740960437
This pull request/issue has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: correct loggly syslog emergency severity [apisix]
Posted by "moying2026 (via GitHub)" <gi...@apache.org>.
moying2026 commented on PR #13390:
URL: https://github.com/apache/apisix/pull/13390#issuecomment-4685460545
👋 Friendly ping — this PR has been open for a while without any review activity. Would any maintainer be able to take a look when they have a chance? Thank you!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: correct loggly syslog emergency severity [apisix]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] closed pull request #13390: fix: correct loggly syslog emergency severity
URL: https://github.com/apache/apisix/pull/13390
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org