Skip to content
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

Add Actuator Health Checking #808

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

RoboticsAndCloud
Copy link

您好,elunez,

我是RoboticsAndCloud(Github昵称),最近在学习Spring-Boot, 在Github中找到这个优秀的项目ELADMIN,浏览了下您的个人网站,学习膜拜大佬之余,也想成为一名Contributor为其做些贡献。

我Fork了代码并线下添加了Actuator功能,以及后面计划使用React为其更新下前端。
本次功能:"Actuator“健康检查功能,自己测试通过.
1)添加 依赖 在pom.xml文件中
2)application-dev.yaml增加Actuator配置
3) 在eladmin-system/src/main/java/me/zhengjie/config/ConfigurerAdapter.java添加注册endpoint逻辑
4)在SpringSecurityConfig.java开放访问权限限制
5)附件有详细的功能和配置代码说明,还有本次部署代码过程中的一些小问题,可以添加到项目的在线帮助文档中
6)代码是基于自己的理解,如果有不妥之处,请大佬指点
Spring Boot 运维管理平台(eladmin).pdf

@elunez
Copy link
Owner

elunez commented Jul 26, 2023

收到,等空闲的时候我看看

@RoboticsAndCloud
Copy link
Author

增加了Prometheus 的支持,Spring-Boot 服务可以暴露指标让Prometheus拉取,这次改动稍有些复杂,等Actuator功能被review之后再重新提交Prometheus的相关功能的代码和文档

…openmetrics-text and text/plain,As the Promethuse server sends requests along with the header application/openmetrics-text, But it needs the data with the header text/plain when it receives the responses
@RoboticsAndCloud
Copy link
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants