Skip to content

Update email field length in annotation (User class in security section) #9402

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.

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

Closed
wants to merge 1 commit into from
Closed

Conversation

Holicz
Copy link
Contributor

@Holicz Holicz commented Mar 7, 2018

I believe 60 characters for email is not enough, because maximum length of email is 254 chars. See https://stackoverflow.com/a/574698/3000493

I believe 60 characters for email is not enough, because maximum length of email is 254 chars. See https://stackoverflow.com/a/574698/3000493
@Holicz Holicz changed the title Update email field length in annotation Update email field length in annotation (User class in security section) Mar 7, 2018
@javiereguiluz javiereguiluz modified the milestones: 2.2, 2.7 Mar 7, 2018
@javiereguiluz
Copy link
Member

A minor but nice change! Thanks. We merged it on 2.7 branch because that's the oldest maintained branch (https://symfony.com/roadmap)

javiereguiluz added a commit that referenced this pull request Mar 7, 2018
…curity section) (Holicz)

This PR was submitted for the 4.0 branch but it was merged into the 2.7 branch instead (closes #9402).

Discussion
----------

Update email field length in annotation (User class in security section)

I believe 60 characters for email is not enough, because maximum length of email is 254 chars. See https://stackoverflow.com/a/574698/3000493

Commits
-------

8324c93 Update email field length in annotation
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.

3 participants