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

Length of Admin Email for GO Server #695

Open
dave-ingham-tlr opened this Issue Nov 3, 2014 · 2 comments

Comments

Projects
None yet
5 participants
@dave-ingham-tlr

dave-ingham-tlr commented Nov 3, 2014

I am establishing the "Email Notification" section of the Server Configuration for the GO server via the web user interface.

I have found that the Admin Email address appears to be truncated at 65 characters. The address that I need to supply is 80 characters long and is therefore being rejected as the truncated address is not valid.

Is this a hard limit to the length or can it be increased?

@arvindsv

This comment has been minimized.

Show comment
Hide comment
@arvindsv

arvindsv Nov 3, 2014

Member

Hmm, you're right. Max length seems to be 64, for no apparent reason. The
limit doesn't seem to be a hard limit. I was able to use the config XML to
change it to:
abcdefghijklmnopqrstuvwxyz_abcdefghijklmnopqrstuvwxyz0123456789_abcdefghijkl@gmail.com,
which has more than 64 characters. Can you try that?

Cheers,
Aravind

On Mon, Nov 3, 2014 at 10:41 AM, dave-ingham-tlr notifications@github.com
wrote:

I am establishing the "Email Notification" section of the Server
Configuration for the GO server via the web user interface.

I have found that the Admin Email address appears to be truncated at 65
characters. The address that I need to supply is 80 characters long and is
therefore being rejected as the truncated address is not valid.

Is this a hard limit to the length or can it be increased?


Reply to this email directly or view it on GitHub
#695.

Member

arvindsv commented Nov 3, 2014

Hmm, you're right. Max length seems to be 64, for no apparent reason. The
limit doesn't seem to be a hard limit. I was able to use the config XML to
change it to:
abcdefghijklmnopqrstuvwxyz_abcdefghijklmnopqrstuvwxyz0123456789_abcdefghijkl@gmail.com,
which has more than 64 characters. Can you try that?

Cheers,
Aravind

On Mon, Nov 3, 2014 at 10:41 AM, dave-ingham-tlr notifications@github.com
wrote:

I am establishing the "Email Notification" section of the Server
Configuration for the GO server via the web user interface.

I have found that the Admin Email address appears to be truncated at 65
characters. The address that I need to supply is 80 characters long and is
therefore being rejected as the truncated address is not valid.

Is this a hard limit to the length or can it be increased?


Reply to this email directly or view it on GitHub
#695.

@dave-ingham-tlr

This comment has been minimized.

Show comment
Hide comment
@dave-ingham-tlr

dave-ingham-tlr Nov 6, 2014

Thanks. This workaround worked. I was also able to go back into the Server Configuration page and send a test email successfully.

dave-ingham-tlr commented Nov 6, 2014

Thanks. This workaround worked. I was also able to go back into the Server Configuration page and send a test email successfully.

@jyotisingh jyotisingh added level1 and removed level1 labels Nov 11, 2014

@zabil zabil removed the level 1 label Jun 3, 2015

@arvindsv arvindsv modified the milestone: Unknown - Needs interest from someone Jan 4, 2016

tedops pushed a commit to tedops/gocd that referenced this issue Jun 6, 2016

arvindsv added a commit that referenced this issue Jun 6, 2016

Merge pull request #2269 from tedops/master
#695 Fixing admin email length to fit RFC 3696, Errata 1003

tedops pushed a commit to tedops/gocd that referenced this issue Jun 6, 2016

tedops pushed a commit to tedops/gocd that referenced this issue Jun 6, 2016

arvindsv added a commit that referenced this issue Jun 6, 2016

Merge pull request #2271 from tedops/master
'From' field fix - #695 Fixing admin email length to fit RFC 3696, Er…

arvindsv added a commit to arvindsv/gocd that referenced this issue Jun 7, 2016

tedops pushed a commit to tedops/gocd that referenced this issue Jun 7, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment