{"action":"created","issue":{"url":"https://api.github.com/repos/symfony/recipes/issues/462","repository_url":"https://api.github.com/repos/symfony/recipes","labels_url":"https://api.github.com/repos/symfony/recipes/issues/462/labels{/name}","comments_url":"https://api.github.com/repos/symfony/recipes/issues/462/comments","events_url":"https://api.github.com/repos/symfony/recipes/issues/462/events","html_url":"https://github.com/symfony/recipes/pull/462","id":360257477,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE1NTU3NDE2","number":462,"title":"Move more platform-dependent options to the DSN","user":{"login":"scaytrase","id":6578413,"node_id":"MDQ6VXNlcjY1Nzg0MTM=","avatar_url":"https://avatars2.githubusercontent.com/u/6578413?v=4","gravatar_id":"","url":"https://api.github.com/users/scaytrase","html_url":"https://github.com/scaytrase","followers_url":"https://api.github.com/users/scaytrase/followers","following_url":"https://api.github.com/users/scaytrase/following{/other_user}","gists_url":"https://api.github.com/users/scaytrase/gists{/gist_id}","starred_url":"https://api.github.com/users/scaytrase/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/scaytrase/subscriptions","organizations_url":"https://api.github.com/users/scaytrase/orgs","repos_url":"https://api.github.com/users/scaytrase/repos","events_url":"https://api.github.com/users/scaytrase/events{/privacy}","received_events_url":"https://api.github.com/users/scaytrase/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-09-14T10:57:48Z","updated_at":"2018-09-14T11:36:28Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/symfony/recipes/pulls/462","html_url":"https://github.com/symfony/recipes/pull/462","diff_url":"https://github.com/symfony/recipes/pull/462.diff","patch_url":"https://github.com/symfony/recipes/pull/462.patch"},"body":"| Q             | A\r\n| ------------- | ---\r\n| License       | MIT\r\n\r\nhttps://github.com/doctrine/dbal/blob/a2cdd8a5b6f507e7f0c509fb9b0134561db357c8/lib/Doctrine/DBAL/DriverManager.php#L215-L272\r\n\r\nIIUC this code, doctrine/dbal already supports parsing extra options from DSN query string.\r\n\r\nI suggest move platform-dependent options to the DSN by default, making it easier, i.e to switch on sqlite for testing just switching the DSN without leaving confusing 5.7 version and driver setting behinf\r\n"},"comment":{"url":"https://api.github.com/repos/symfony/recipes/issues/comments/421331928","html_url":"https://github.com/symfony/recipes/pull/462#issuecomment-421331928","issue_url":"https://api.github.com/repos/symfony/recipes/issues/462","id":421331928,"node_id":"MDEyOklzc3VlQ29tbWVudDQyMTMzMTkyOA==","user":{"login":"scaytrase","id":6578413,"node_id":"MDQ6VXNlcjY1Nzg0MTM=","avatar_url":"https://avatars2.githubusercontent.com/u/6578413?v=4","gravatar_id":"","url":"https://api.github.com/users/scaytrase","html_url":"https://github.com/scaytrase","followers_url":"https://api.github.com/users/scaytrase/followers","following_url":"https://api.github.com/users/scaytrase/following{/other_user}","gists_url":"https://api.github.com/users/scaytrase/gists{/gist_id}","starred_url":"https://api.github.com/users/scaytrase/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/scaytrase/subscriptions","organizations_url":"https://api.github.com/users/scaytrase/orgs","repos_url":"https://api.github.com/users/scaytrase/repos","events_url":"https://api.github.com/users/scaytrase/events{/privacy}","received_events_url":"https://api.github.com/users/scaytrase/received_events","type":"User","site_admin":false},"created_at":"2018-09-14T11:36:28Z","updated_at":"2018-09-14T11:36:28Z","author_association":"NONE","body":"@BoShurik I didn't remove the default env which this comment belongs to. If config parameters are requried - could we do something like `DATABASE_URL: mysql://localhost/?server_version=5.7` as env default (or whatever the default values are), which will make this more explicit"}}
{"actor":{"display_login":"scaytrase"}}