SAP-samples / SAP-samples/ams-samples-java
docs: Javalin sample README claims AMS authorization is an unimplemented TODO, but it is fully implemented
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Java
- Sterne
- 0
- Forks
- 0
- Ø Merge
- 2 T. 15 Std.
- Gemergte PRs (30 T.)
- 5
Beschreibung
ams-javalin-shopping/README.md currently claims authorization isn't implemented [1]. Instead of (as claimed) a stub, the authorization is implemented in AuthHandler.java [2].
[1] https://github.com/SAP-samples/ams-samples-java/blob/main/ams-javalin-shopping/README.md#L114-L123
[2] https://github.com/SAP-samples/ams-samples-java/blob/main/ams-javalin-shopping/src/main/java/com/sap/cloud/security/ams/samples/auth/AuthHandler.java
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Lies den Abschnitt zur Autorisierung in ams-javalin-shopping/README.md und vergleiche seine Aussagen mit ams-javalin-shopping/src/main/java/com/sap/cloud/security/ams/samples/auth/AuthHandler.java. Aktualisiere die README so, dass sie die implementierte Autorisierung korrekt wiedergibt und keine Aussage mehr enthält, dass sie ein nicht implementiertes TODO ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- documentation
- Issue-Typ
- Dokumentation
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- Unter einer Stunde
- Aktivitätsstatus
- Ruhig
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 78/100