Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-70239

Case sensitivity on emails

XMLWordPrintable

      If an E-Mail-Address of a author appears once as 'theauthor@mailto.me' and once as 'TheAuthor@mailto.me' is get counts as two authors instead of one.

          [JENKINS-70239] Case sensitivity on emails

          Tim Schlüter created issue -
          Ulli Hafner made changes -
          Assignee Original: Ulli Hafner [ drulli ]
          Ulli Hafner made changes -
          Labels New: help-wanted newbie-friendly
          Jagruti made changes -
          Comment [ Hey ]

          Jagruti added a comment - - edited

          Hi,

          I found the email being set here.

          The issue says that the email ids are case sensitive.

          To make them insensitive, do I have to set the email id in say lowercase only?

          My solution would be: 

          lineBlame.setEmail(emailByLine.get(line).toLowerCase());

          I am not really aware of the code base.

          I would appreciate your guidance here.

          Jagruti added a comment - - edited Hi, I found the email being set here. The issue says that the email ids are case sensitive. To make them insensitive, do I have to set the email id in say lowercase only? My solution would be:  lineBlame.setEmail(emailByLine.get(line).toLowerCase()); I am not really aware of the code base. I would appreciate your guidance here.
          Jagruti made changes -
          Assignee New: Jagruti [ jagrutiti ]

          Ulli Hafner added a comment -

          Your solution will help only partially. A fix is much simpler:

          https://github.com/jenkinsci/forensics-api-plugin/blob/master/src/main/java/io/jenkins/plugins/forensics/miner/CommitStatistics.java#L188

          This line will count the number of different authors. This needs to be made case insensitive.

          Ulli Hafner added a comment - Your solution will help only partially. A fix is much simpler: https://github.com/jenkinsci/forensics-api-plugin/blob/master/src/main/java/io/jenkins/plugins/forensics/miner/CommitStatistics.java#L188 This line will count the number of different authors. This needs to be made case insensitive.

          Ulli Hafner added a comment -

          timsch Where exactly did you see the problem? Only in the commit statistics table? Or are there different places?

          Ulli Hafner added a comment - timsch Where exactly did you see the problem? Only in the commit statistics table? Or are there different places?

          Udit added a comment -

          Is this bug resolved or is still open?

          Udit added a comment - Is this bug resolved or is still open?

          Ulli Hafner added a comment -

          The bug is still open, I'm not sure if jagrutiti still plans to work on it.

          Ulli Hafner added a comment - The bug is still open, I'm not sure if jagrutiti still plans to work on it.

            jagrutiti Jagruti
            timsch Tim Schlüter
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: