apache / apache/cloudstack

remove instanceof checks and use reflection for admin cmd invocations

Đang mở
#6,966 5 bình luận 0 reaction 0 người được giao Xem trên GitHub
no-issue-activity status:needs-work status:stale type:technical-debt
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

Mở 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

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.