{"action":"created","issue":{"url":"https://api.github.com/repos/UNINETT/mod_auth_mellon/issues/26","labels_url":"https://api.github.com/repos/UNINETT/mod_auth_mellon/issues/26/labels{/name}","comments_url":"https://api.github.com/repos/UNINETT/mod_auth_mellon/issues/26/comments","events_url":"https://api.github.com/repos/UNINETT/mod_auth_mellon/issues/26/events","html_url":"https://github.com/UNINETT/mod_auth_mellon/issues/26","id":68623877,"number":26,"title":"Put number of multi-valued values to separate variable","user":{"login":"adelton","id":2536912,"avatar_url":"https://avatars.githubusercontent.com/u/2536912?v=3","gravatar_id":"","url":"https://api.github.com/users/adelton","html_url":"https://github.com/adelton","followers_url":"https://api.github.com/users/adelton/followers","following_url":"https://api.github.com/users/adelton/following{/other_user}","gists_url":"https://api.github.com/users/adelton/gists{/gist_id}","starred_url":"https://api.github.com/users/adelton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/adelton/subscriptions","organizations_url":"https://api.github.com/users/adelton/orgs","repos_url":"https://api.github.com/users/adelton/repos","events_url":"https://api.github.com/users/adelton/events{/privacy}","received_events_url":"https://api.github.com/users/adelton/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-04-15T09:18:18Z","updated_at":"2015-04-17T13:54:34Z","closed_at":"2015-04-17T13:54:34Z","body":"When consuming multi-valued attributes in Spacewalk and Foreman (please also see https://github.com/UNINETT/mod_auth_mellon/issues/24), the code expects variable suffixed _N to exist and hold the number of values. So for example, if the user is a member of two groups, and REMOTE_USER_GROUP_1=test1 and REMOTE_USER_GROUP_2=test2 are set, REMOTE_USER_GROUP_N would be set to 2 as well. The behaviour is also mentioned at http://www.freeipa.org/page/Environment_Variables#Proposed_Additional_Variables.\r\nThat makes it easier for the code to know how many values to expect and also plays the role of a safeguard.\r\nI'd like to propose a patch to mod_auth_mellon to be able to populate the _N variable as well. Before I present a code though, would you prefer for it to be set unconditionally, or would you prefer a configuration option for it?"},"comment":{"url":"https://api.github.com/repos/UNINETT/mod_auth_mellon/issues/comments/93990560","html_url":"https://github.com/UNINETT/mod_auth_mellon/issues/26#issuecomment-93990560","issue_url":"https://api.github.com/repos/UNINETT/mod_auth_mellon/issues/26","id":93990560,"user":{"login":"olavmrk","id":349658,"avatar_url":"https://avatars.githubusercontent.com/u/349658?v=3","gravatar_id":"","url":"https://api.github.com/users/olavmrk","html_url":"https://github.com/olavmrk","followers_url":"https://api.github.com/users/olavmrk/followers","following_url":"https://api.github.com/users/olavmrk/following{/other_user}","gists_url":"https://api.github.com/users/olavmrk/gists{/gist_id}","starred_url":"https://api.github.com/users/olavmrk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/olavmrk/subscriptions","organizations_url":"https://api.github.com/users/olavmrk/orgs","repos_url":"https://api.github.com/users/olavmrk/repos","events_url":"https://api.github.com/users/olavmrk/events{/privacy}","received_events_url":"https://api.github.com/users/olavmrk/received_events","type":"User","site_admin":false},"created_at":"2015-04-17T13:54:34Z","updated_at":"2015-04-17T13:54:34Z","body":"Patch merged -- closing issue."}}