Skip to content
This repository was archived by the owner on Sep 17, 2021. It is now read-only.

Conversation

@supertom
Copy link
Contributor

DisplayName is an optional field. Changed to Email for consistency. This was raised in Gitter chat, but I don't think it was officially reported.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 60.414% when pulling 0839c50 on supertom:watcher-fix into 370f40e on Netflix:develop.

@scriptsrc
Copy link
Contributor

Is there a max length on email?

Item.name is limited to 303 chars:
https://github.com/Netflix/security_monkey/blob/develop/security_monkey/datastore.py#L226

@scriptsrc scriptsrc merged commit 7d484b2 into Netflix:develop Apr 26, 2017
@scriptsrc scriptsrc mentioned this pull request May 24, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants