{"action":"opened","issue":{"url":"https://api.github.com/repos/snc/SncRedisBundle/issues/306","repository_url":"https://api.github.com/repos/snc/SncRedisBundle","labels_url":"https://api.github.com/repos/snc/SncRedisBundle/issues/306/labels{/name}","comments_url":"https://api.github.com/repos/snc/SncRedisBundle/issues/306/comments","events_url":"https://api.github.com/repos/snc/SncRedisBundle/issues/306/events","html_url":"https://github.com/snc/SncRedisBundle/issues/306","id":193561130,"number":306,"title":"make client options a variableNode","user":{"login":"ivoba","id":471254,"avatar_url":"https://avatars.githubusercontent.com/u/471254?v=3","gravatar_id":"","url":"https://api.github.com/users/ivoba","html_url":"https://github.com/ivoba","followers_url":"https://api.github.com/users/ivoba/followers","following_url":"https://api.github.com/users/ivoba/following{/other_user}","gists_url":"https://api.github.com/users/ivoba/gists{/gist_id}","starred_url":"https://api.github.com/users/ivoba/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ivoba/subscriptions","organizations_url":"https://api.github.com/users/ivoba/orgs","repos_url":"https://api.github.com/users/ivoba/repos","events_url":"https://api.github.com/users/ivoba/events{/privacy}","received_events_url":"https://api.github.com/users/ivoba/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-12-05T17:30:59Z","updated_at":"2016-12-05T17:30:59Z","closed_at":null,"body":"How about replacing the fix client options with a free variableNode?\r\n\r\nIt seems the fix options are limiting the clients since not every option is correctly represented or changes in different redis/predis version are covered.\r\n\r\nThe Guzzle bundle does this and it works well, imho:\r\nhttps://github.com/csarrazi/CsaGuzzleBundle/blob/master/src/DependencyInjection/Configuration.php#L118\r\n\r\nCurrently these issues are related to limitations of the bundle configuration, mostly in cluster context:\r\nhttps://github.com/snc/SncRedisBundle/issues/284\r\nhttps://github.com/snc/SncRedisBundle/issues/267\r\nhttps://github.com/snc/SncRedisBundle/pull/305\r\nhttps://github.com/snc/SncRedisBundle/pull/274\r\nhttps://github.com/snc/SncRedisBundle/pull/199\r\n\r\nSo this could make configuration more flexible.\r\n\r\nA downside would be the missing validation and default values for the options, but i guess there can be a workaround for this and imho its ok to leave a correct option configuration to the users responsibility.\r\n\r\nThat freedom & responsibility thing ... ;)\r\n\r\n@snc what do you think?\r\nam i missing something here?"}}
{"actor":{"display_login":"ivoba"}}