Uploaded image for project: 'SonarQube Plugins'
  1. SonarQube Plugins
  2. SONARPLUGINS-2877

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

    Details

      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
        jeanbaptiste 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
        jeanbaptiste 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 Freddy Mallet added a comment -

        Manually tested !

        Show
        freddy.mallet Freddy Mallet added a comment - Manually tested !
        Hide
        jeanbaptiste 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
        jeanbaptiste 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
        jeanbaptiste Jean-Baptiste Vilain added a comment -

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

        Show
        jeanbaptiste 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 Freddy Mallet added a comment -

        Manually tested !

        Show
        freddy.mallet Freddy Mallet added a comment - Manually tested !
        Hide
        jeanbaptiste Jean-Baptiste Vilain added a comment -

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

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

          People

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

            Dates

            • Created:
              Updated:
              Resolved: