Re: [tomcat] branch main updated: Add support for additional user attributes in TomcatPrincipal

2022-02-13 Thread Rémy Maucherat
tbox.apache.org/repos/asf/tomcat.git > > > > > > The following commit(s) were added to refs/heads/main by this push: > > new c3edf43 Add support for additional user attributes in > > TomcatPrincipal > > c3edf43 is described below > > > > commit c3edf437da

Re: Add support for additional user attributes in TomcatPrincipal

2022-02-11 Thread Rémy Maucherat
On Fri, Feb 11, 2022 at 11:17 AM Carsten Klein wrote: > > > > > On Fri, Feb 11, 2022 at 10:10 AM Rémy Maucherat wrote > > > > If we get there, it could include mail addresses, ssn, payment info, > > user profile pictures (binary), etc etc. > > Also one thing I don't get now is how this became Obje

Re: [tomcat] branch main updated: Add support for additional user, attributes in TomcatPrincipal

2022-02-11 Thread Michael Osipov
Am 2022-02-11 um 11:16 schrieb Rémy Maucherat: On Fri, Feb 11, 2022 at 10:47 AM Michael Osipov wrote: I cannot share your further explanations as well. The Realm *is* already an object store. Principal name and roles are attributes on that object, so is password. Without this possibility you ha

Re: Add support for additional user attributes in TomcatPrincipal

2022-02-11 Thread Carsten Klein
On Fri, Feb 11, 2022 at 10:10 AM Rémy Maucherat wrote If we get there, it could include mail addresses, ssn, payment info, user profile pictures (binary), etc etc. Also one thing I don't get now is how this became Object getAttribute(String name) instead of String getAttribute(String name) ?

Re: Re: [tomcat] branch main updated: Add support for additional user, attributes in TomcatPrincipal

2022-02-11 Thread Rémy Maucherat
On Fri, Feb 11, 2022 at 10:47 AM Michael Osipov wrote: > > Rémy, > > really, I don't understand your work attitude. Carsten contacted us on > the mailing list before providing the PR. Then he provided the PR. It > was open for EIGHT months. We discussed this in and out, with you and > Mark. Everyo

Re: Add support for additional user attributes in TomcatPrincipal

2022-02-11 Thread Michael Osipov
On Fri, Feb 11, 2022 at 8:10 AM Carsten Klein wrote: Hello R=C3=A9my, Mark and Michael, I'm new to the dev list and did not get your recent mails and didn't figure out how to get them in order to answer. So, I decided to start a new thread (sorry for that). I guess, having those attributes wi

Re: Re: [tomcat] branch main updated: Add support for additional user, attributes in TomcatPrincipal

2022-02-11 Thread Michael Osipov
Rémy, really, I don't understand your work attitude. Carsten contacted us on the mailing list before providing the PR. Then he provided the PR. It was open for EIGHT months. We discussed this in and out, with you and Mark. Everyone expressed the conditions he is willing to accept the PR. I ha

Re: Add support for additional user attributes in TomcatPrincipal

2022-02-11 Thread Rémy Maucherat
On Fri, Feb 11, 2022 at 8:10 AM Carsten Klein wrote: > > Hello Rémy, Mark and Michael, > > I'm new to the dev list and did not get your recent mails and didn't > figure out how to get them in order to answer. So, I decided to start a > new thread (sorry for that). > > I guess, having those attribu

Add support for additional user attributes in TomcatPrincipal

2022-02-10 Thread Carsten Klein
Hello Rémy, Mark and Michael, I'm new to the dev list and did not get your recent mails and didn't figure out how to get them in order to answer. So, I decided to start a new thread (sorry for that). I guess, having those attributes with the Principal (aka Principal metadata) does not make t

Re: [tomcat] branch main updated: Add support for additional user attributes in TomcatPrincipal

2022-02-10 Thread Rémy Maucherat
a commit to branch main > >> in repository https://gitbox.apache.org/repos/asf/tomcat.git > >> > >> > >> The following commit(s) were added to refs/heads/main by this push: > >> new c3edf43 Add support for additio

Re: [tomcat] branch main updated: Add support for additional user attributes in TomcatPrincipal

2022-02-10 Thread Mark Thomas
refs/heads/main by this push: new c3edf43 Add support for additional user attributes in TomcatPrincipal c3edf43 is described below commit c3edf437da20af0f11edc0ad6d893399b01e6287 Author: Carsten Klein AuthorDate: Wed Jan 12 15:06:42 2022 +0100 Add support for additional user attributes

Re: [tomcat] branch main updated: Add support for additional user attributes in TomcatPrincipal

2022-02-10 Thread Rémy Maucherat
/heads/main by this push: > new c3edf43 Add support for additional user attributes in > TomcatPrincipal > c3edf43 is described below > > commit c3edf437da20af0f11edc0ad6d893399b01e6287 > Author: Carsten Klein > AuthorDate: Wed Jan 12 15:06:42 2022 +0100 > > Add suppor

[tomcat] branch 9.0.x updated: Add support for additional user attributes in TomcatPrincipal

2022-02-09 Thread michaelo
attributes in TomcatPrincipal 024a495 is described below commit 024a4956f0053569beb3f842dc1154c9b7979977 Author: Carsten Klein AuthorDate: Tue Feb 8 16:53:22 2022 +0100 Add support for additional user attributes in TomcatPrincipal This closes #472 --- java/org/apache/catalina

[tomcat] branch 10.0.x updated: Add support for additional user attributes in TomcatPrincipal

2022-02-09 Thread michaelo
attributes in TomcatPrincipal 8ced4ec is described below commit 8ced4ec586696db64639de08dad3293611737e32 Author: Carsten Klein AuthorDate: Wed Jan 12 15:06:42 2022 +0100 Add support for additional user attributes in TomcatPrincipal This closes #471 --- java/org/apache/catalina

[tomcat] branch main updated: Add support for additional user attributes in TomcatPrincipal

2022-02-08 Thread michaelo
attributes in TomcatPrincipal c3edf43 is described below commit c3edf437da20af0f11edc0ad6d893399b01e6287 Author: Carsten Klein AuthorDate: Wed Jan 12 15:06:42 2022 +0100 Add support for additional user attributes in TomcatPrincipal This closes #463 --- java/org/apache/catalina