Backend: Add the availiable userDetails when using the AD-Header approach about hajk HOT 4 OPEN

from hajk.

To be honest, I'm not sure. It feels like the possible outcome of a getMap request and the structure of the userDetails object will differ a lot between implementations. E.g. here's how it looks on one AD setup:

Perhaps we should come up with a common interface for this object?

from hajk.

One thing that comes into mind: if you active to show the button with user initials, what does it say in your setup?

I'm wondering because I notice that I'm utilising the displayName field, as well as the description field, both of which are lacking in your screenshot. Instead you have some fields that I haven't seen before.

As stated previously, I think we should agree upon some interface for userDetails. Perhaps sAMAccountName, mail, displayName and description should always be available?

from hajk.

Actually I have never seen the "button with user initials".

Well, that would be nice. But it would mean even more changes to NodeHoster and even more data will be send on every request through the proxy. Then it will need to be handled in nodebackend as well.

As I cant prioritize this right now and it's not urgent, I would prefer more additions to userDetails as a new issue in both Hajk-repo and NodeHoster-repo. What do you think?

from hajk.