remove instanceof checks and use reflection for admin cmd invocations
- Ngôn ngữ chính
- Java
- Star
- 3.1k
- Fork
- 1.4k
- Merge trung bình
- 6 ngày 19 giờ
- Pull request đã merge (30 ngày)
- 32
Mô tả
##### ISSUE TYPE
* cleanup
##### COMPONENT NAME
~~~
API
~~~
##### CLOUDSTACK VERSION
~~~
4.17
~~~
##### CONFIGURATION
##### OS / ENVIRONMENT
##### SUMMARY
the API classes have instances that are for root admin only, these are checked for using `instanceof` in the service layer. This should be cleaned up in favour for reflection invocations as in #6951. It would be better to do all in one go but the fix in #6951 really needs it and is for a point release, so postponing.
##### STEPS TO REPRODUCE
~~~
~~~
##### EXPECTED RESULTS
~~~
~~~
##### ACTUAL RESULTS
~~~
~~~
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu từ các lớp API và lớp dịch vụ xử lý việc gọi các lệnh admin, sau đó so sánh với cách tiếp cận reflection được mô tả trong #6951. Truy vết các kiểm tra instanceof hiện có đối với các instance chỉ dành cho root admin và xác định các đường dẫn gọi bị ảnh hưởng. Hoàn tất khi các kiểm tra đó sử dụng cách tiếp cận reflection được định hướng, đồng thời giữ nguyên hành vi chỉ dành cho admin.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- java
- Lĩnh vực
- api, cloud
- Loại issue
- Tái cấu trúc
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Ít trao đổi
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 45/100