[
https://jira.duraspace.org/browse/DS-337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=26928#comment-26928
]
Ivan Masár commented on DS-337:
-------------------------------
I would recommend against trimming the email address and instead recommend
checking conformance of the email address to the appropriate RFCs to avoid not
just this one issue, but more subtle issues in the future.
http://en.wikipedia.org/wiki/Email_address#Syntax
Surely Java must already have an implementation.
Here's what a quick search gave me:
http://stackoverflow.com/questions/624581/what-is-the-best-java-email-address-validation-method
> A bug related with adding new -EPersons
> ---------------------------------------
>
> Key: DS-337
> URL: https://jira.duraspace.org/browse/DS-337
> Project: DSpace
> Issue Type: Bug
> Components: DSpace API
> Reporter: Eija Airio
> Assignee: Claudia Jürgen
> Fix For: 1.6.0
>
>
> When the system admin is adding new E-Persons, and types a space character
> after the email address, the space is stored to the email column in the
> EPerson table as well. This naturally causes problems: for example the same
> user is able to register with an email address without the space characters.
> This can be corrected by adding a trim funcion in
> dspace-xmlui/dspace-xmlui-api/src/main/java/org/dspace/app/xmlui/aspect/administrative/FlowEPersonUtils.java:
> String email = (request.getParameter("email_address")).trim();
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
------------------------------------------------------------------------------
Monitor your physical, virtual and cloud infrastructure from a single
web console. Get in-depth insight into apps, servers, databases, vmware,
SAP, cloud infrastructure, etc. Download 30-day Free Trial.
Pricing starts from $795 for 25 servers or applications!
http://p.sf.net/sfu/zoho_dev2dev_nov
_______________________________________________
Dspace-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dspace-devel
