SonarQube Plugins
  1. SonarQube Plugins
  2. SONARPLUGINS-2877

With Git, use the user email instead of user name to feed Sonar

    Details

    • Number of attachments :
      0

      Description

      Otherwise any non-ASCII character can be used and using such string as a key in the DB can lead to some unexpected behavior as soon as the encoding of the DB is not the UTF-8 one.

        Activity

        Hide
        Jean-Baptiste Vilain added a comment -

        The plugin will now use the user email as identifier when Git is the scm provider.
        This issue also upgrades the dependency to maven scm api from 1.7 to 1.8.

        Show
        Jean-Baptiste Vilain added a comment - The plugin will now use the user email as identifier when Git is the scm provider. This issue also upgrades the dependency to maven scm api from 1.7 to 1.8.
        Hide
        Freddy Mallet added a comment -

        Manually tested !

        Show
        Freddy Mallet added a comment - Manually tested !
        Hide
        Jean-Baptiste Vilain added a comment -

        The existing ITs have been updated to cover this new behavior when using Git as the SCM provider.

        Show
        Jean-Baptiste Vilain added a comment - The existing ITs have been updated to cover this new behavior when using Git as the SCM provider.
        Hide
        Jean-Baptiste Vilain added a comment -

        Issue reopened since unexpected behaviour has been observed: in some cases, the extracted email contains a trailing ">".

        Show
        Jean-Baptiste Vilain added a comment - Issue reopened since unexpected behaviour has been observed: in some cases, the extracted email contains a trailing ">".
        Hide
        Freddy Mallet added a comment -

        Manually tested !

        Show
        Freddy Mallet added a comment - Manually tested !
        Hide
        Jean-Baptiste Vilain added a comment -

        This last fix has been unit-tested and does not require a new IT.

        Show
        Jean-Baptiste Vilain added a comment - This last fix has been unit-tested and does not require a new IT.

          People

          • Assignee:
            Jean-Baptiste Vilain
            Reporter:
            Freddy Mallet
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: