Skip to content

bc(contrib/log/aliyun): aliyun sls logger #3622

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.

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

Merged
merged 1 commit into from
Apr 21, 2025
Merged

Conversation

aveyuan
Copy link
Contributor

@aveyuan aveyuan commented Apr 10, 2025

Fix aliyun sls logger

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Apr 10, 2025
@aveyuan
Copy link
Contributor Author

aveyuan commented Apr 21, 2025

@shenqidebaozi 大佬,合并一下,阿里云之前的logger适配根本用不了,api也做了变更,无法正常工作。

@kratos-ci-bot
Copy link
Collaborator

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


@shenqidebaozi Big boss, let’s merge it. Alibaba Cloud’s previous logger adaptation is not available at all, and the API has also been changed and cannot work normally.

@dosubot dosubot bot added the LGTM label Apr 21, 2025
@shenqidebaozi shenqidebaozi changed the title fix(contrib/log/aliyun): aliyun sls logger bc(contrib/log/aliyun): aliyun sls logger Apr 21, 2025
@shenqidebaozi shenqidebaozi merged commit 1c3e0c9 into go-kratos:main Apr 21, 2025
50 of 65 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LGTM size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants