{"action":"created","issue":{"url":"https://api.github.com/repos/symfony/symfony/issues/10342","labels_url":"https://api.github.com/repos/symfony/symfony/issues/10342/labels{/name}","comments_url":"https://api.github.com/repos/symfony/symfony/issues/10342/comments","events_url":"https://api.github.com/repos/symfony/symfony/issues/10342/events","html_url":"https://github.com/symfony/symfony/pull/10342","id":28412560,"number":10342,"title":"[SecurityBundle] Allow 'have to match pattern' paths in AbstractFactory children configuration","user":{"login":"yguedidi","id":1480128,"avatar_url":"https://avatars.githubusercontent.com/u/1480128?v=3","gravatar_id":"","url":"https://api.github.com/users/yguedidi","html_url":"https://github.com/yguedidi","followers_url":"https://api.github.com/users/yguedidi/followers","following_url":"https://api.github.com/users/yguedidi/following{/other_user}","gists_url":"https://api.github.com/users/yguedidi/gists{/gist_id}","starred_url":"https://api.github.com/users/yguedidi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yguedidi/subscriptions","organizations_url":"https://api.github.com/users/yguedidi/orgs","repos_url":"https://api.github.com/users/yguedidi/repos","events_url":"https://api.github.com/users/yguedidi/events{/privacy}","received_events_url":"https://api.github.com/users/yguedidi/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/symfony/symfony/labels/Security","name":"Security","color":"DDDDDD"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":14,"created_at":"2014-02-27T11:45:29Z","updated_at":"2015-02-11T15:19:07Z","closed_at":"2015-02-11T14:41:09Z","pull_request":{"url":"https://api.github.com/repos/symfony/symfony/pulls/10342","html_url":"https://github.com/symfony/symfony/pull/10342","diff_url":"https://github.com/symfony/symfony/pull/10342.diff","patch_url":"https://github.com/symfony/symfony/pull/10342.patch"},"body":"| Q             | A\r\n| ------------- | ---\r\n| Bug fix?      | no\r\n| New feature?  | yes\r\n| BC breaks?    | no\r\n| Deprecations? | no\r\n| Tests pass?   | yes\r\n| Fixed tickets | no\r\n| License       | MIT\r\n| Doc PR        | no\r\n\r\nThis PR allow security factories that extend AbstractFactory to define paths that have to match the firewall pattern (like `check_path`) in configuration.\r\n\r\nWhat do you think about it ?\r\n\r\n**TODO**\r\n- [x] Add tests"},"comment":{"url":"https://api.github.com/repos/symfony/symfony/issues/comments/73898090","html_url":"https://github.com/symfony/symfony/pull/10342#issuecomment-73898090","issue_url":"https://api.github.com/repos/symfony/symfony/issues/10342","id":73898090,"user":{"login":"yguedidi","id":1480128,"avatar_url":"https://avatars.githubusercontent.com/u/1480128?v=3","gravatar_id":"","url":"https://api.github.com/users/yguedidi","html_url":"https://github.com/yguedidi","followers_url":"https://api.github.com/users/yguedidi/followers","following_url":"https://api.github.com/users/yguedidi/following{/other_user}","gists_url":"https://api.github.com/users/yguedidi/gists{/gist_id}","starred_url":"https://api.github.com/users/yguedidi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yguedidi/subscriptions","organizations_url":"https://api.github.com/users/yguedidi/orgs","repos_url":"https://api.github.com/users/yguedidi/repos","events_url":"https://api.github.com/users/yguedidi/events{/privacy}","received_events_url":"https://api.github.com/users/yguedidi/received_events","type":"User","site_admin":false},"created_at":"2015-02-11T15:19:07Z","updated_at":"2015-02-11T15:19:07Z","body":"Allowing only the (hardcoded) `check_path` option in the abstract factory looks weird to me, that's why I send this PR."}}