remove instanceof checks and use reflection for admin cmd invocations
- 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
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