{"action":"created","issue":{"url":"https://api.github.com/repos/GluuFederation/oxTrust/issues/1257","repository_url":"https://api.github.com/repos/GluuFederation/oxTrust","labels_url":"https://api.github.com/repos/GluuFederation/oxTrust/issues/1257/labels{/name}","comments_url":"https://api.github.com/repos/GluuFederation/oxTrust/issues/1257/comments","events_url":"https://api.github.com/repos/GluuFederation/oxTrust/issues/1257/events","html_url":"https://github.com/GluuFederation/oxTrust/issues/1257","id":363343172,"node_id":"MDU6SXNzdWUzNjMzNDMxNzI=","number":1257,"title":"Issues when adding several redirect_uri for an OIDC client's entry","user":{"login":"aliaksander-samuseu","id":11802046,"node_id":"MDQ6VXNlcjExODAyMDQ2","avatar_url":"https://avatars0.githubusercontent.com/u/11802046?v=4","gravatar_id":"","url":"https://api.github.com/users/aliaksander-samuseu","html_url":"https://github.com/aliaksander-samuseu","followers_url":"https://api.github.com/users/aliaksander-samuseu/followers","following_url":"https://api.github.com/users/aliaksander-samuseu/following{/other_user}","gists_url":"https://api.github.com/users/aliaksander-samuseu/gists{/gist_id}","starred_url":"https://api.github.com/users/aliaksander-samuseu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aliaksander-samuseu/subscriptions","organizations_url":"https://api.github.com/users/aliaksander-samuseu/orgs","repos_url":"https://api.github.com/users/aliaksander-samuseu/repos","events_url":"https://api.github.com/users/aliaksander-samuseu/events{/privacy}","received_events_url":"https://api.github.com/users/aliaksander-samuseu/received_events","type":"User","site_admin":false},"labels":[{"id":199659949,"node_id":"MDU6TGFiZWwxOTk2NTk5NDk=","url":"https://api.github.com/repos/GluuFederation/oxTrust/labels/High%20Priority","name":"High Priority","color":"b60205","default":false},{"id":88928175,"node_id":"MDU6TGFiZWw4ODkyODE3NQ==","url":"https://api.github.com/repos/GluuFederation/oxTrust/labels/bug","name":"bug","color":"fc2929","default":true}],"state":"closed","locked":false,"assignee":{"login":"yurem","id":6104200,"node_id":"MDQ6VXNlcjYxMDQyMDA=","avatar_url":"https://avatars1.githubusercontent.com/u/6104200?v=4","gravatar_id":"","url":"https://api.github.com/users/yurem","html_url":"https://github.com/yurem","followers_url":"https://api.github.com/users/yurem/followers","following_url":"https://api.github.com/users/yurem/following{/other_user}","gists_url":"https://api.github.com/users/yurem/gists{/gist_id}","starred_url":"https://api.github.com/users/yurem/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yurem/subscriptions","organizations_url":"https://api.github.com/users/yurem/orgs","repos_url":"https://api.github.com/users/yurem/repos","events_url":"https://api.github.com/users/yurem/events{/privacy}","received_events_url":"https://api.github.com/users/yurem/received_events","type":"User","site_admin":false},"assignees":[{"login":"yurem","id":6104200,"node_id":"MDQ6VXNlcjYxMDQyMDA=","avatar_url":"https://avatars1.githubusercontent.com/u/6104200?v=4","gravatar_id":"","url":"https://api.github.com/users/yurem","html_url":"https://github.com/yurem","followers_url":"https://api.github.com/users/yurem/followers","following_url":"https://api.github.com/users/yurem/following{/other_user}","gists_url":"https://api.github.com/users/yurem/gists{/gist_id}","starred_url":"https://api.github.com/users/yurem/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yurem/subscriptions","organizations_url":"https://api.github.com/users/yurem/orgs","repos_url":"https://api.github.com/users/yurem/repos","events_url":"https://api.github.com/users/yurem/events{/privacy}","received_events_url":"https://api.github.com/users/yurem/received_events","type":"User","site_admin":false},{"login":"syntrydy","id":7513418,"node_id":"MDQ6VXNlcjc1MTM0MTg=","avatar_url":"https://avatars3.githubusercontent.com/u/7513418?v=4","gravatar_id":"","url":"https://api.github.com/users/syntrydy","html_url":"https://github.com/syntrydy","followers_url":"https://api.github.com/users/syntrydy/followers","following_url":"https://api.github.com/users/syntrydy/following{/other_user}","gists_url":"https://api.github.com/users/syntrydy/gists{/gist_id}","starred_url":"https://api.github.com/users/syntrydy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/syntrydy/subscriptions","organizations_url":"https://api.github.com/users/syntrydy/orgs","repos_url":"https://api.github.com/users/syntrydy/repos","events_url":"https://api.github.com/users/syntrydy/events{/privacy}","received_events_url":"https://api.github.com/users/syntrydy/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/GluuFederation/oxTrust/milestones/22","html_url":"https://github.com/GluuFederation/oxTrust/milestone/22","labels_url":"https://api.github.com/repos/GluuFederation/oxTrust/milestones/22/labels","id":3145420,"node_id":"MDk6TWlsZXN0b25lMzE0NTQyMA==","number":22,"title":"3.1.4","description":"","creator":{"login":"willow9886","id":5271048,"node_id":"MDQ6VXNlcjUyNzEwNDg=","avatar_url":"https://avatars0.githubusercontent.com/u/5271048?v=4","gravatar_id":"","url":"https://api.github.com/users/willow9886","html_url":"https://github.com/willow9886","followers_url":"https://api.github.com/users/willow9886/followers","following_url":"https://api.github.com/users/willow9886/following{/other_user}","gists_url":"https://api.github.com/users/willow9886/gists{/gist_id}","starred_url":"https://api.github.com/users/willow9886/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/willow9886/subscriptions","organizations_url":"https://api.github.com/users/willow9886/orgs","repos_url":"https://api.github.com/users/willow9886/repos","events_url":"https://api.github.com/users/willow9886/events{/privacy}","received_events_url":"https://api.github.com/users/willow9886/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":201,"state":"open","created_at":"2018-02-26T21:49:55Z","updated_at":"2018-09-26T09:04:16Z","due_on":null,"closed_at":null},"comments":4,"created_at":"2018-09-24T22:26:40Z","updated_at":"2018-09-26T09:04:16Z","closed_at":"2018-09-26T09:04:16Z","author_association":"NONE","body":"Environment:\r\ngluu-server-3.1.4-RC6, CentOS7\r\n\r\nSteps to reproduce;\r\n1. Log in to web UI\r\n2. Move to \"OpenID Connect -> Clients\"\r\n3. Create a new client entry with minimal set of required properties; use \"pairwise\" for \"Subject Type\" and specify only one redirect login uri\r\n4. Try to add an additional redirect uri **with the same hostname part** as the 1st one has (that step should fail, see below for details)\r\n5. Supply a valid \"Sector Identifier URI\" which returns a JSON file containing both of the redirect uris used in this test (use a uri pointing to a remote web server), and click the \"Update\" button\r\n6. Try to add 2nd redirect uri again\r\n\r\nResults:\r\n1. At step 4) an error message is displayed reading \"A sector identifier must be defined first\" and Java error trace is registered in `oxtrust.log` (full text is [here](https://github.com/GluuFederation/oxTrust/files/2413132/sector_id_uri_issue.txt)):\r\n```\r\n2018-09-24 16:44:07,520 ERROR [qtp1094834071-14] [org.gluu.oxtrust.ldap.service.SectorIdentifierService] (SectorIdentifierService.java:90) - Failed to find sector identifier by oxId sector-uris.json\r\norg.gluu.site.ldap.persistence.exception.EntryPersistenceException: Failed to find entry: oxId=sector-uris.json,ou=sector_identifiers,o=@!E8DE.7456.204B.BE5E!0001!25AB.4068,o=gluu\r\n\tat org.gluu.site.ldap.persistence.LdapEntryManager.find(LdapEntryManager.java:306) ~[oxcore-ldap-3.1.4-SNAPSHOT.jar:?]\r\n\tat org.gluu.site.ldap.persistence.AbstractEntryManager.find(AbstractEntryManager.java:444) ~[oxcore-ldap-3.1.4-SNAPSHOT.jar:?]\r\n\tat org.gluu.site.ldap.persistence.AbstractEntryManager.find(AbstractEntryManager.java:381) ~[oxcore-ldap-3.1.4-SNAPSHOT.jar:?]\r\n```\r\n2. At step 6) the same message and same log record are seen again, despite of the fact \"Sector Identifier URI\" is defined now\r\n\r\nExpected results:\r\n1. According [to the spec](https://openid.net/specs/openid-connect-core-1_0.html#PairwiseAlg), sector identifier uri only becomes mandatory when several redirect uris are used and their hostname parts are different. Thus the fact a 2nd uri couldn't been added at step 4) doesn't seem right\r\n2. Even when a correct sector identifier uri is added as web UI requests, it's still not possible to add a 2nd redirect uri, thus it doesn't seem this web UI control functions properly at all\r\n"},"comment":{"url":"https://api.github.com/repos/GluuFederation/oxTrust/issues/comments/424641557","html_url":"https://github.com/GluuFederation/oxTrust/issues/1257#issuecomment-424641557","issue_url":"https://api.github.com/repos/GluuFederation/oxTrust/issues/1257","id":424641557,"node_id":"MDEyOklzc3VlQ29tbWVudDQyNDY0MTU1Nw==","user":{"login":"syntrydy","id":7513418,"node_id":"MDQ6VXNlcjc1MTM0MTg=","avatar_url":"https://avatars3.githubusercontent.com/u/7513418?v=4","gravatar_id":"","url":"https://api.github.com/users/syntrydy","html_url":"https://github.com/syntrydy","followers_url":"https://api.github.com/users/syntrydy/followers","following_url":"https://api.github.com/users/syntrydy/following{/other_user}","gists_url":"https://api.github.com/users/syntrydy/gists{/gist_id}","starred_url":"https://api.github.com/users/syntrydy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/syntrydy/subscriptions","organizations_url":"https://api.github.com/users/syntrydy/orgs","repos_url":"https://api.github.com/users/syntrydy/repos","events_url":"https://api.github.com/users/syntrydy/events{/privacy}","received_events_url":"https://api.github.com/users/syntrydy/received_events","type":"User","site_admin":false},"created_at":"2018-09-26T09:04:16Z","updated_at":"2018-09-26T09:04:16Z","author_association":"CONTRIBUTOR","body":"@aliaksander-samuseu  i think it's okay now."}}
{"actor":{"display_login":"syntrydy"}}