Skip to content

Conversation

@xabbuh
Copy link
Member

@xabbuh xabbuh commented Nov 27, 2015

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #16704
License MIT
Doc PR

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing newline

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and I think we add the legacy annotation anyway

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with @Tobion

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, may help us find it in the master branch.

@xabbuh xabbuh force-pushed the pr-16704-legacy-test branch from f481307 to 444b6ef Compare November 28, 2015 08:35
@nicolas-grekas
Copy link
Member

Thank you @xabbuh.

@nicolas-grekas nicolas-grekas merged commit 444b6ef into symfony:2.8 Nov 28, 2015
nicolas-grekas added a commit that referenced this pull request Nov 28, 2015
…(xabbuh)

This PR was merged into the 2.8 branch.

Discussion
----------

[SecurityBundle] test legacy CSRF configuration options

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #16704
| License       | MIT
| Doc PR        |

Commits
-------

444b6ef test legacy CSRF configuration options
@xabbuh xabbuh deleted the pr-16704-legacy-test branch November 28, 2015 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants