{"action":"created","issue":{"url":"https://api.github.com/repos/clarin-eric/SPF-SPs-metadata/issues/50","repository_url":"https://api.github.com/repos/clarin-eric/SPF-SPs-metadata","labels_url":"https://api.github.com/repos/clarin-eric/SPF-SPs-metadata/issues/50/labels{/name}","comments_url":"https://api.github.com/repos/clarin-eric/SPF-SPs-metadata/issues/50/comments","events_url":"https://api.github.com/repos/clarin-eric/SPF-SPs-metadata/issues/50/events","html_url":"https://github.com/clarin-eric/SPF-SPs-metadata/pull/50","id":344771765,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA0MDg1Njk5","number":50,"title":"Exchanged the DARIAH AAI connector, since DARIAH now works with a AAI…","user":{"login":"martinhaase","id":5486396,"node_id":"MDQ6VXNlcjU0ODYzOTY=","avatar_url":"https://avatars1.githubusercontent.com/u/5486396?v=4","gravatar_id":"","url":"https://api.github.com/users/martinhaase","html_url":"https://github.com/martinhaase","followers_url":"https://api.github.com/users/martinhaase/followers","following_url":"https://api.github.com/users/martinhaase/following{/other_user}","gists_url":"https://api.github.com/users/martinhaase/gists{/gist_id}","starred_url":"https://api.github.com/users/martinhaase/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martinhaase/subscriptions","organizations_url":"https://api.github.com/users/martinhaase/orgs","repos_url":"https://api.github.com/users/martinhaase/repos","events_url":"https://api.github.com/users/martinhaase/events{/privacy}","received_events_url":"https://api.github.com/users/martinhaase/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2018-07-26T09:46:07Z","updated_at":"2018-07-27T15:06:52Z","closed_at":"2018-07-26T13:37:11Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/clarin-eric/SPF-SPs-metadata/pulls/50","html_url":"https://github.com/clarin-eric/SPF-SPs-metadata/pull/50","diff_url":"https://github.com/clarin-eric/SPF-SPs-metadata/pull/50.diff","patch_url":"https://github.com/clarin-eric/SPF-SPs-metadata/pull/50.patch"},"body":"… proxy model. \r\nAccording to DFN Metadata there is now a new xmlns:remd=\"http://refeds.org/metadata\", I just added it to the CLARIN Metadata. Feel free to remove it, together with the according contact type. \r\nRegards, Martin\r\n"},"comment":{"url":"https://api.github.com/repos/clarin-eric/SPF-SPs-metadata/issues/comments/408447101","html_url":"https://github.com/clarin-eric/SPF-SPs-metadata/pull/50#issuecomment-408447101","issue_url":"https://api.github.com/repos/clarin-eric/SPF-SPs-metadata/issues/50","id":408447101,"node_id":"MDEyOklzc3VlQ29tbWVudDQwODQ0NzEwMQ==","user":{"login":"andmor-","id":6597509,"node_id":"MDQ6VXNlcjY1OTc1MDk=","avatar_url":"https://avatars0.githubusercontent.com/u/6597509?v=4","gravatar_id":"","url":"https://api.github.com/users/andmor-","html_url":"https://github.com/andmor-","followers_url":"https://api.github.com/users/andmor-/followers","following_url":"https://api.github.com/users/andmor-/following{/other_user}","gists_url":"https://api.github.com/users/andmor-/gists{/gist_id}","starred_url":"https://api.github.com/users/andmor-/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andmor-/subscriptions","organizations_url":"https://api.github.com/users/andmor-/orgs","repos_url":"https://api.github.com/users/andmor-/repos","events_url":"https://api.github.com/users/andmor-/events{/privacy}","received_events_url":"https://api.github.com/users/andmor-/received_events","type":"User","site_admin":false},"created_at":"2018-07-27T15:06:52Z","updated_at":"2018-07-27T15:06:52Z","author_association":"CONTRIBUTOR","body":"OK since this is not really a CLARIN SPF SP and the metadata is only kept here due to the absence of a better place, I will proceed with the merge and make sure this is not pushed to DFN-AAI or eduGAIN.\r\n\r\nSome remarks about the components:\r\n\r\n> DARIAH Services authenticating with CLARIN IdPs - this is covered since both our proxy is in eduGAIN, and you are working hard to put your IdP in eduGAIN. Our proxy has CoCo and R&S enabled, which are the current standards DARIAH chose to follow.\r\n\r\nThis is only partially true. \r\nKeep in ind there are no \"CLARIN IdPs\" (apart from the homeless one). What we have is CLARIN countries. Their federations and institutions manage their IdPs which usually are not related to anything specific to CLARIN. All we do is to aggregate the metadata of the IdPs from these countries and republish it in a single file so our SPs can easily consume it.\r\nIn this scope, no user from the CLARIN country e.g. Netherlands will be able to use her own institution's IdP to login into DARIAH Services. Their IdPs are typically not in eduGAIN and when they are, they still require an explicit authorization from the IdP admin to connect to any SP for the first time.\r\nCLARIN does not try in any way to change the way the national identity federations work with eduGAIN. Nor SURF (in this example) is willing to change their policy.\r\n\r\n> CLARIN services authenticating with DARIAH IdPs - this is covered since you are working hard to put these services in eduGAIN, and all DARIAH IdPs are in eduGAIN, including the DARIAH homeless IdP.\r\n\r\nUnder which registration authority is he DARIAH homeless IdP published on eduGAIN? It could be that we are not importing it into our feed.\r\nAssuming all DARIAH IdPs consume all CLARIN SPF metadata via eduGAIN there is still a problem: if a  \"DARIAH IdP\" uses whitelisting (very typical in e.g. Germany) or is member of an hub and spoke federation (e.g. the Netherlands) this won't work for that IdP.\r\nAlso where are these IdPs located? By default, our SPs only consume eduGAIN IdP metadata from CLARIN countries.\r\n\r\n> DARIAH Services authenticating with the CLARIN homeless IdP - not yet covered, since your IdP is not in eduGAIN; this is all this pull request is about.\r\n\r\nMerging...\r\n\r\n\r\n \r\n\r\n"}}
{"actor":{"display_login":"andmor-"}}