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.

Sign up for GitHub

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

Jump to bottom

[Enhancement] RESTful API framework for EventMeshAdmin #4494

Closed
3 tasks done
Pil0tXia opened this issue Oct 19, 2023 · 0 comments · Fixed by #4498
Closed
3 tasks done

[Enhancement] RESTful API framework for EventMeshAdmin #4494

Pil0tXia opened this issue Oct 19, 2023 · 0 comments · Fixed by #4498
Labels
enhancement New feature or request

Comments

@Pil0tXia
Copy link
Member

Search before asking

  • I had searched in the issues and found no similar issues.

Enhancement Request

Optimize the EventMeshAdmin API into a RESTful style to enhance its capability to provide detailed error messages and HTTP status codes to the caller, enable the propagation of error messages as exceptions, and support the extension of error message types and usage.

Describe the solution you"d like

  • Errors: An enumeration class that conforms to the RESTful specifications and custom error reporting requirements.
  • Result: A RESTful response DTO.
  • BaseException: Customized error reporting using enums and exceptions
  • GlobalExceptionHandler: handle BaseException

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct
@Pil0tXia Pil0tXia added the enhancement New feature or request label Oct 19, 2023
@Pil0tXia Pil0tXia mentioned this issue Oct 19, 2023
Merged
xwm1992 pushed a commit that referenced this issue Oct 20, 2023
@Pil0tXia
[ISSUE #4494] RESTful API framework for EventMeshAdmin (#4498)
62935c4 
* restful response optimize part1

* restful response optimize part2

* restful response optimize part3

* restful response optimize part4

* exception handling

* exception handling part2

* exception handling part3

* exception handling part4

* exception handling part5

* add error type to error displaying

* warp json message response instead of string
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ISSUE #4494] RESTful API framework for EventMeshAdmin Pil0tXia/eventmesh
1 participant
@Pil0tXia