)]}'
{"deployment/keystone/keystone-container-puppet.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"53a5c5732bf3083c1b404507ef2c45545b432279","unresolved":true,"context_lines":[{"line_number":366,"context_line":"      The delimiter to use when setting multi-valued claims."},{"line_number":367,"context_line":"  KeystoneOpenIdcPassUserInfoAs:"},{"line_number":368,"context_line":"    type: string"},{"line_number":369,"context_line":"    default: \u0027claims\u0027"},{"line_number":370,"context_line":"    description: \u003e-"},{"line_number":371,"context_line":"      Define the way(s) in which the claims resolved from the userinfo endpoint"},{"line_number":372,"context_line":"      are passed to the application according to OIDCPassClaimsAs."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"cb9bba63_8899e9ba","line":369,"range":{"start_line":369,"start_character":14,"end_line":369,"end_character":20},"updated":"2021-08-11 05:50:55.000000000","message":"It\u0027d be useful if we can implement validation at tht layer using allowed_values, rather than let the deployment fail later during puppet apply.","commit_id":"06694b38c64c7ed261ad0a982d0670577d0e15d4"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"53a5c5732bf3083c1b404507ef2c45545b432279","unresolved":true,"context_lines":[{"line_number":374,"context_line":"    type: string"},{"line_number":375,"context_line":"    default: \u0027both\u0027"},{"line_number":376,"context_line":"    description: \u003e-"},{"line_number":377,"context_line":"      Define the way in which the claims and tokens are passed to the application environment:"},{"line_number":378,"context_line":"      \"none\": no claims/tokens are passed"},{"line_number":379,"context_line":"      \"environment\": claims/tokens are passed as environment variables"},{"line_number":380,"context_line":"      \"headers\": claims/tokens are passed in headers (also useful in reverse proxy scenario\u0027s)"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"772052ae_7db747b2","line":377,"range":{"start_line":377,"start_character":6,"end_line":377,"end_character":94},"updated":"2021-08-11 05:50:55.000000000","message":"Ditto","commit_id":"06694b38c64c7ed261ad0a982d0670577d0e15d4"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"0358ff8e55b0edf8fd6f96c1469292292aa19deb","unresolved":true,"context_lines":[{"line_number":604,"context_line":"                get_param: KeystoneOpenIdcPassUserInfoAs"},{"line_number":605,"context_line":"              keystone::federation::openidc::openidc_pass_claim_as:"},{"line_number":606,"context_line":"                get_param: KeystoneOpenIdcPassClaimsAs"},{"line_number":607,"context_line":"              keystone::federation::openidc::openidc_claim_delimiter:"},{"line_number":608,"context_line":"                get_param: KeystoneOpenIdcClaimDelimiter"},{"line_number":609,"context_line":"              keystone::federation::openidc::openidc_cache_type:"},{"line_number":610,"context_line":"                if:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"bd43e497_248c9d1d","line":607,"range":{"start_line":607,"start_character":0,"end_line":607,"end_character":68},"updated":"2021-09-16 07:24:17.000000000","message":"it doesn\u0027t seem to exist in the depends-on patch? Care to explain how it\u0027s used?","commit_id":"3f91911314681743497c18afad833f229a9f7924"},{"author":{"_account_id":30073,"name":"Brendan Shephard","email":"bshephar@bne-home.net","username":"bshephar"},"change_message_id":"be53a5156a1b4d2dcbf4eea3b2454c2736089f7b","unresolved":true,"context_lines":[{"line_number":604,"context_line":"                get_param: KeystoneOpenIdcPassUserInfoAs"},{"line_number":605,"context_line":"              keystone::federation::openidc::openidc_pass_claim_as:"},{"line_number":606,"context_line":"                get_param: KeystoneOpenIdcPassClaimsAs"},{"line_number":607,"context_line":"              keystone::federation::openidc::openidc_claim_delimiter:"},{"line_number":608,"context_line":"                get_param: KeystoneOpenIdcClaimDelimiter"},{"line_number":609,"context_line":"              keystone::federation::openidc::openidc_cache_type:"},{"line_number":610,"context_line":"                if:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"09f1fe27_87c1cf9a","line":607,"range":{"start_line":607,"start_character":0,"end_line":607,"end_character":68},"in_reply_to":"bd43e497_248c9d1d","updated":"2021-09-16 07:43:06.000000000","message":"Hey, this one already existed prior to my patch. I\u0027m just exposing it via THT:\nhttps://github.com/openstack/puppet-keystone/blob/master/manifests/federation/openidc.pp#L145","commit_id":"3f91911314681743497c18afad833f229a9f7924"}]}