Found that fuse adapter doesn't work due to adapters packaging changes. JIRA is here: https://issues.jboss.org/browse/KEYCLOAK-2816 Among some minor things, the biggest issue is that osgi doesn't work properly if there are 2 modules with classes inside same package. There is "adapter-spi" and "adapter-spi-public" modules with both having classes inside package "org.keycloak.adapters.spi". So the easiest solution was just to rename the package. There was possibility to rename either: (1) the one inside "adapter-spi" (2) or the one inside "adapter-spi-public" . I've actually chose the (1) and renamed the one inside "adapter-spi" to "org.keycloak.adapters.spi .internal". Only reason is that there might be some users, who are using classes AuthenticationError or LogoutError, so they don't need to change the code of their applications because upgrade. On the other hand the (2) (rename the package inside "adapter-spi-public" to "org.keycloak.adapters.spi.public" and keep the one inside "adapter-spi" unrenamed) have the advantage of better consistency among module names and package names. So if you rather want me to go this way, let me know and I can change it. For now, the PR against 1.9.x for path1 is here: https://github.com/keycloak/keycloak/pull/2614 . There are 72 affected files because of package renamed, but all of them are inside adapters. Will send PR against master too if you agree with the path1. Not sure if we rather need to retest all the examples because of the change... :-( Marek _______________________________________________ keycloak-dev mailing list keycloak-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/keycloak-dev

Rename or merge either works for me. We need it resolved asap though. Yes, we're going to have to re-test all adapters and examples because of this :( On 13 April 2016 at 18:18, Bill Burke <bburke(a)redhat.com <mailto:bburke@redhat.com>> wrote: Wait! Maybe we should just merge adapter-spi and adapter-spi-public. Its not that big a deal to have adapter-spi be public. On 4/13/2016 12:10 PM, Marek Posolda wrote: > Found that fuse adapter doesn't work due to adapters packaging > changes. JIRA is here: https://issues.jboss.org/browse/KEYCLOAK-2816 > > Among some minor things, the biggest issue is that osgi doesn't > work properly if there are 2 modules with classes inside same > package. There is "adapter-spi" and "adapter-spi-public" modules > with both having classes inside package > "org.keycloak.adapters.spi". So the easiest solution was just to > rename the package. > > There was possibility to rename either: > (1) the one inside "adapter-spi" > (2) or the one inside "adapter-spi-public" . > > I've actually chose the (1) and renamed the one inside > "adapter-spi" to "org.keycloak.adapters.spi .internal". Only > reason is that there might be some users, who are using classes > AuthenticationError or LogoutError, so they don't need to change > the code of their applications because upgrade. On the other hand > the (2) (rename the package inside "adapter-spi-public" to > "org.keycloak.adapters.spi.public" and keep the one inside > "adapter-spi" unrenamed) have the advantage of better consistency > among module names and package names. So if you rather want me to > go this way, let me know and I can change it. > > For now, the PR against 1.9.x for path1 is here: > https://github.com/keycloak/keycloak/pull/2614 . There are 72 > affected files because of package renamed, but all of them are > inside adapters. Will send PR against master too if you agree > with the path1. Not sure if we rather need to retest all the > examples because of the change... :-( > > Marek > > > _______________________________________________ > keycloak-dev mailing list > keycloak-dev(a)lists.jboss.org <mailto:keycloak-dev@lists.jboss.org> > https://lists.jboss.org/mailman/listinfo/keycloak-dev -- Bill Burke JBoss, a division of Red Hat http://bill.burkecentral.com _______________________________________________ keycloak-dev mailing list keycloak-dev(a)lists.jboss.org <mailto:keycloak-dev@lists.jboss.org> https://lists.jboss.org/mailman/listinfo/keycloak-dev _______________________________________________ keycloak-dev mailing list keycloak-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/keycloak-dev

I'll look into it tonight after I get home from driving kid around. On 4/13/2016 2:10 PM, Marek Posolda wrote: > Ok, I've closed the https://github.com/keycloak/keycloak/pull/2614 > > Do you want me to look at merging modules or are you looking at it > Bill? I can sort it, but will be able to start on it no earlier than > tomorrow morning Europe time :/ > > Marek > > On 13/04/16 20:03, Stian Thorgersen wrote: >> Rename or merge either works for me. We need it resolved asap >> though. Yes, we're going to have to re-test all adapters and >> examples because of this :( >> >> On 13 April 2016 at 18:18, Bill Burke <bburke(a)redhat.com >> <mailto:bburke@redhat.com>> wrote: >> >> Wait! Maybe we should just merge adapter-spi and >> adapter-spi-public. Its not that big a deal to have adapter-spi >> be public. >> >> >> On 4/13/2016 12:10 PM, Marek Posolda wrote: >>> Found that fuse adapter doesn't work due to adapters packaging >>> changes. JIRA is here: >>> https://issues.jboss.org/browse/KEYCLOAK-2816 >>> >>> Among some minor things, the biggest issue is that osgi doesn't >>> work properly if there are 2 modules with classes inside same >>> package. There is "adapter-spi" and "adapter-spi-public" >>> modules with both having classes inside package >>> "org.keycloak.adapters.spi". So the easiest solution was just >>> to rename the package. >>> >>> There was possibility to rename either: >>> (1) the one inside "adapter-spi" >>> (2) or the one inside "adapter-spi-public" . >>> >>> I've actually chose the (1) and renamed the one inside >>> "adapter-spi" to "org.keycloak.adapters.spi .internal". Only >>> reason is that there might be some users, who are using classes >>> AuthenticationError or LogoutError, so they don't need to >>> change the code of their applications because upgrade. On the >>> other hand the (2) (rename the package inside >>> "adapter-spi-public" to "org.keycloak.adapters.spi.public" and >>> keep the one inside "adapter-spi" unrenamed) have the advantage >>> of better consistency among module names and package names. So >>> if you rather want me to go this way, let me know and I can >>> change it. >>> >>> For now, the PR against 1.9.x for path1 is here: >>> https://github.com/keycloak/keycloak/pull/2614 . There are 72 >>> affected files because of package renamed, but all of them are >>> inside adapters. Will send PR against master too if you agree >>> with the path1. Not sure if we rather need to retest all the >>> examples because of the change... :-( >>> >>> Marek >>> >>> >>> _______________________________________________ >>> keycloak-dev mailing list >>> keycloak-dev(a)lists.jboss.org <mailto:keycloak-dev@lists.jboss.org> >>> https://lists.jboss.org/mailman/listinfo/keycloak-dev >> >> -- >> Bill Burke >> JBoss, a division of Red Hat >> http://bill.burkecentral.com >> >> >> _______________________________________________ >> keycloak-dev mailing list >> keycloak-dev(a)lists.jboss.org <mailto:keycloak-dev@lists.jboss.org> >> https://lists.jboss.org/mailman/listinfo/keycloak-dev >> >> >> >> >> _______________________________________________ >> keycloak-dev mailing list >> keycloak-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/keycloak-dev > -- Bill Burke JBoss, a division of Red Hat http://bill.burkecentral.com