JCommuneIcon indicating the project type

JCommune

Uploaded image for project: 'JCommune'
  1. JCommune
  2. JC-1945

Jcommune not encrypts passwords before sending registration request to poulpe

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.10 Larks
    • Fix Version/s: 2.10 Larks
    • Labels:
      None
    • Sprint:
      2.10 Larks

      Description

      Preconditions:
      1. Poulpe auth plugin should be enabled
      2. Poulpe and Jcommune should work with different databases
      Steps to reproduce:
      1. Open forum as anonymous
      2. Press "Sign up" button
      3. Enter correct user data
      4. Press "Sing up" button
      5. Find newly created user in poulpe database

      AR: user unencrypted user password stored
      ER: md5 hash of password stored

        Attachments

          Issue Links

          Discovered while testing

          Bug - A problem which impairs or prevents the functions of the product. JC-1850 Error while user try to login via poulpe

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            mixaS Mikhail Stryzhonak added a comment -

            Fixed

            Show
            mixaS Mikhail Stryzhonak added a comment - Fixed
            Hide
            Permalink
            jenkins Jenkins Bot added a comment -

            SUCCESS: Integrated in JC. Unit Tests #2451
            #JC-1945:Fixed (mixas.27: 10803e3e01f092a2f8fbb63aea6a7f0412d4e80e)

            Show
            jenkins Jenkins Bot added a comment - SUCCESS: Integrated in JC. Unit Tests #2451 # JC-1945 :Fixed (mixas.27: 10803e3e01f092a2f8fbb63aea6a7f0412d4e80e )
            Hide
            Permalink
            ctapobep Stanislav Bashkyrtsev added a comment -

            Please, write tests that cover this case - they should specifically check that hash is sent instead of clear password.

            Show
            ctapobep Stanislav Bashkyrtsev added a comment - Please, write tests that cover this case - they should specifically check that hash is sent instead of clear password.
            Hide
            Permalink
            jenkins Jenkins Bot added a comment -

            SUCCESS: Integrated in JC. Unit Tests #2453
            #JC-1945: Added unit test and javadoc (mixas.27: 845217d1b0049c8e21a69c66dbdb50a722441f83)

            Show
            jenkins Jenkins Bot added a comment - SUCCESS: Integrated in JC. Unit Tests #2453 # JC-1945 : Added unit test and javadoc (mixas.27: 845217d1b0049c8e21a69c66dbdb50a722441f83 )
            Hide
            Permalink
            jenkins Jenkins Bot added a comment -

            UNSTABLE: Integrated in JC. Unit Tests #2454
            #JC-1945: Added javadoc (mixas.27: 19dff6bb1dfd82fd28901d44e22f89742ad5dff1)

            Show
            jenkins Jenkins Bot added a comment - UNSTABLE: Integrated in JC. Unit Tests #2454 # JC-1945 : Added javadoc (mixas.27: 19dff6bb1dfd82fd28901d44e22f89742ad5dff1 )
            Hide
            Permalink
            ctapobep Stanislav Bashkyrtsev added a comment - - edited

            Let's discuss the test in the evening

            Show
            ctapobep Stanislav Bashkyrtsev added a comment - - edited Let's discuss the test in the evening
            Hide
            Permalink
            jenkins Jenkins Bot added a comment -

            SUCCESS: Integrated in JC. Unit Tests #2455
            #JC-1945: Fixed language issues (mixas.27: 676e8cd96d699d48f4857b90161fde54751ced66)

            Show
            jenkins Jenkins Bot added a comment - SUCCESS: Integrated in JC. Unit Tests #2455 # JC-1945 : Fixed language issues (mixas.27: 676e8cd96d699d48f4857b90161fde54751ced66 )
            Hide
            Permalink
            jenkins Jenkins Bot added a comment -

            SUCCESS: Integrated in JC. Unit Tests #2456
            #JC-1945: Added package-info (mixas.27: 49ee7b84a9abf7e485f96559d1e3407d4fd986c7)

            Show
            jenkins Jenkins Bot added a comment - SUCCESS: Integrated in JC. Unit Tests #2456 # JC-1945 : Added package-info (mixas.27: 49ee7b84a9abf7e485f96559d1e3407d4fd986c7 )
            Hide
            Permalink
            ctapobep Stanislav Bashkyrtsev added a comment -

            Moving to QA. Unit tests don't block QA testing.

            Show
            ctapobep Stanislav Bashkyrtsev added a comment - Moving to QA. Unit tests don't block QA testing.
            Hide
            Permalink
            QAPasha Paul Panaitov added a comment -

            Environment: JCommune 2.10.2475, Poulpe DB, JCommune DB, Firefox 30.0

            Test Scenario
            https://drive.google.com/file/d/0B8yHOKAkzcivSEhuSDQ1d25CT00/edit?usp=sharing

            Result: issue can be closed.

            Show
            QAPasha Paul Panaitov added a comment - Environment: JCommune 2.10.2475, Poulpe DB, JCommune DB, Firefox 30.0 Test Scenario https://drive.google.com/file/d/0B8yHOKAkzcivSEhuSDQ1d25CT00/edit?usp=sharing Result: issue can be closed.

              People

              • Assignee:
                QAPasha Paul Panaitov
                Reporter:
                mixaS Mikhail Stryzhonak
              • Votes:
                0 Vote for this issue
                Watchers:
                Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Structure Helper Panel