Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Move SecurityUserValueResolver to security-http #25588

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Dec 31, 2017
Merged

Move SecurityUserValueResolver to security-http #25588

merged 1 commit into from
Dec 31, 2017

Conversation

chalasr
Copy link
Member

@chalasr chalasr commented Dec 23, 2017

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? yes
Tests pass? yes
Fixed tickets n/a
License MIT
Doc PR n/a

@chalasr chalasr added this to the 4.1 milestone Dec 23, 2017
@chalasr chalasr force-pushed the move_userresolver_security branch from 8e6acfb to 30a07e7 Compare December 23, 2017 00:51
@linaori
Copy link
Contributor

linaori commented Dec 23, 2017

Makes sense 👍

@fabpot
Copy link
Member

fabpot commented Dec 31, 2017

Thank you @chalasr.

@fabpot fabpot merged commit 30a07e7 into symfony:master Dec 31, 2017
fabpot added a commit that referenced this pull request Dec 31, 2017
@fabpot
feature #25588 Move SecurityUserValueResolver to security-http (chalasr)
8c33cd4 
This PR was merged into the 4.1-dev branch.

Discussion
----------

Move SecurityUserValueResolver to security-http

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | yes
| Tests pass?   | yes
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

Commits
-------

30a07e7 Move SecurityUserValueResolver to security-http
@chalasr chalasr deleted the move_userresolver_security branch December 31, 2017 04:15
@javiereguiluz javiereguiluz mentioned this pull request Jan 8, 2018
Closed
@fabpot fabpot mentioned this pull request May 7, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fabpot fabpot fabpot approved these changes

@nicolas-grekas nicolas-grekas nicolas-grekas approved these changes

Assignees
No one assigned
Labels
Deprecation Feature Security SecurityBundle Status: Reviewed
Projects
None yet
Milestone
4.1
Development

Successfully merging this pull request may close these issues.
5 participants
@chalasr @linaori @fabpot @nicolas-grekas @carsonbot