{"action":"created","issue":{"url":"https://api.github.com/repos/apereo/cas/issues/1674","repository_url":"https://api.github.com/repos/apereo/cas","labels_url":"https://api.github.com/repos/apereo/cas/issues/1674/labels{/name}","comments_url":"https://api.github.com/repos/apereo/cas/issues/1674/comments","events_url":"https://api.github.com/repos/apereo/cas/issues/1674/events","html_url":"https://github.com/apereo/cas/issues/1674","id":145890946,"number":1674,"title":"Upgrade PersonDirectory to 1.7.1: allow ldapPersonAttributeDao's resultAttributeMapping entry keys to be case-insensitive","user":{"login":"jerrykan","id":377632,"avatar_url":"https://avatars.githubusercontent.com/u/377632?v=3","gravatar_id":"","url":"https://api.github.com/users/jerrykan","html_url":"https://github.com/jerrykan","followers_url":"https://api.github.com/users/jerrykan/followers","following_url":"https://api.github.com/users/jerrykan/following{/other_user}","gists_url":"https://api.github.com/users/jerrykan/gists{/gist_id}","starred_url":"https://api.github.com/users/jerrykan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jerrykan/subscriptions","organizations_url":"https://api.github.com/users/jerrykan/orgs","repos_url":"https://api.github.com/users/jerrykan/repos","events_url":"https://api.github.com/users/jerrykan/events{/privacy}","received_events_url":"https://api.github.com/users/jerrykan/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/apereo/cas/labels/Attribute%20Resolution","name":"Attribute Resolution","color":"5319e7"},{"url":"https://api.github.com/repos/apereo/cas/labels/Bug","name":"Bug","color":"e11d21"},{"url":"https://api.github.com/repos/apereo/cas/labels/Minor","name":"Minor","color":"fbca04"}],"state":"closed","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/apereo/cas/milestones/27","html_url":"https://github.com/apereo/cas/milestones/4.2.2","labels_url":"https://api.github.com/repos/apereo/cas/milestones/27/labels","id":1684513,"number":27,"title":"4.2.2","description":"CAS v4.2.2 patch release","creator":{"login":"mmoayyed","id":1205228,"avatar_url":"https://avatars.githubusercontent.com/u/1205228?v=3","gravatar_id":"","url":"https://api.github.com/users/mmoayyed","html_url":"https://github.com/mmoayyed","followers_url":"https://api.github.com/users/mmoayyed/followers","following_url":"https://api.github.com/users/mmoayyed/following{/other_user}","gists_url":"https://api.github.com/users/mmoayyed/gists{/gist_id}","starred_url":"https://api.github.com/users/mmoayyed/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmoayyed/subscriptions","organizations_url":"https://api.github.com/users/mmoayyed/orgs","repos_url":"https://api.github.com/users/mmoayyed/repos","events_url":"https://api.github.com/users/mmoayyed/events{/privacy}","received_events_url":"https://api.github.com/users/mmoayyed/received_events","type":"User","site_admin":false},"open_issues":2,"closed_issues":22,"state":"open","created_at":"2016-04-05T13:03:25Z","updated_at":"2016-05-10T15:19:05Z","due_on":"2016-05-20T07:00:00Z","closed_at":null},"comments":9,"created_at":"2016-04-05T05:26:58Z","updated_at":"2016-05-14T00:22:39Z","closed_at":"2016-04-13T23:33:56Z","body":"CAS server version: 4.2.1\r\n\r\nThe [LDAP Attribute Resolution](http://jasig.github.io/cas/development/integration/Attribute-Resolution.html#ldap) documentation gives the impression that the `resultAttributeMapping` map entry keys are not case-sensitive, but it appears that only keys with lower case values will work (ie. `<entry key=\"displayName\" value=\"displayName\" />` will not work).\r\n\r\nAs best as I can tell in `org.jasig.services.persondir.support.AbstractQueryPersonAttributeDao.mapPersonAttributes()` the `person` object passed in contains attribute keys that are all lower-case, so the `if (personAttributes.containsKey(dataKey)) {}` condition will only be met if the `dataKey` is also forced to be lower case.\r\n\r\nThough based on the name of the class of the `person` object being passed in (`CaseInsensitiveNamedPersonImpl`) it would seem pointless to create a case insensitive version of the `NamedPersonImpl` if it isn't being used in that way (though I have tried to dig into that part of the code)."},"comment":{"url":"https://api.github.com/repos/apereo/cas/issues/comments/219188322","html_url":"https://github.com/apereo/cas/issues/1674#issuecomment-219188322","issue_url":"https://api.github.com/repos/apereo/cas/issues/1674","id":219188322,"user":{"login":"mmoayyed","id":1205228,"avatar_url":"https://avatars.githubusercontent.com/u/1205228?v=3","gravatar_id":"","url":"https://api.github.com/users/mmoayyed","html_url":"https://github.com/mmoayyed","followers_url":"https://api.github.com/users/mmoayyed/followers","following_url":"https://api.github.com/users/mmoayyed/following{/other_user}","gists_url":"https://api.github.com/users/mmoayyed/gists{/gist_id}","starred_url":"https://api.github.com/users/mmoayyed/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmoayyed/subscriptions","organizations_url":"https://api.github.com/users/mmoayyed/orgs","repos_url":"https://api.github.com/users/mmoayyed/repos","events_url":"https://api.github.com/users/mmoayyed/events{/privacy}","received_events_url":"https://api.github.com/users/mmoayyed/received_events","type":"User","site_admin":false},"created_at":"2016-05-14T00:22:39Z","updated_at":"2016-05-14T00:22:39Z","body":"Schedules are outlined here:\r\nhttps://github.com/apereo/cas/milestones\r\n\r\nYou probably need to exclude the dependency from cas-server-webapp. The sample overlay shows you how to certain files can be excluded from the packaging. "}}