Conversation
0314R
left a comment
0314R
left a comment
There was a problem hiding this comment.
Kleine opmerkingen, verder naar mijn mening een veel elegantere oplossing dan de vorige.
Heb de patch kunnen genereren en ge-applied op een branch vanuit 26.x met dezelfde /src als resultaat als in deze branch.
Gecompiled en getest op onze VPS16 keycloak (ingelogd met eHerkenning-testmiddel en Digid Simulator).
diffgen.sh
Outdated
| PATCH_FILE="./branch-diff.patch" | ||
|
|
||
| FILES=( | ||
| services/src/main/java/org/keycloak/authentication/authenticators/broker/util/SerializedBrokeredIdentityContext.java |
There was a problem hiding this comment.
SerializedBrokeredIdentityContext.java gebruiken we niet meer, daarom gaf dat oude script een error voor .../broker/util
diffgen.sh
Outdated
| fi | ||
|
|
||
| OLD_BRANCH="$1" | ||
| NEW_BRANCH="$2" |
There was a problem hiding this comment.
Als je een random branch neem geeft die niet bestaat, geeft het geen melding en gaat het proces door. Ook als je het script niet in de keycloak repo draait. Misschien mooier als het in die gevallen niets doet
0314R
left a comment
0314R
left a comment
There was a problem hiding this comment.
Als je de comments in mijn vorige review voor deze PR kunt verwerken is dat top, maar je kan ook meteen mergen
2 participants
Patch: origin:release:26.0->origin:archive:release:26.1.patch