apache / apache/cloudstack

remove instanceof checks and use reflection for admin cmd invocations

Offen
#6,966 5 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
no-issue-activity status:needs-work status:stale type:technical-debt
Vorherrschende Sprache
Java
Sterne
3.1k
Forks
1.4k
Ø Merge
6 T. 19 Std.
Gemergte PRs (30 T.)
32

Beschreibung

##### 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

~~~

~~~

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne in den API-Klassen und der Service-Schicht, die Aufrufe von Admin-Befehlen behandeln, und vergleiche anschließend den in #6951 beschriebenen Reflection-Ansatz. Verfolge die vorhandenen instanceof-Prüfungen für Instanzen, die nur für Root-Admins bestimmt sind, und ermittle die betroffenen Aufrufpfade. Die Aufgabe ist abgeschlossen, wenn diese Prüfungen den vorgesehenen Reflection-Ansatz verwenden und dabei das Verhalten nur für Admins beibehalten wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
api, cloud
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.