Project

General

Profile

Actions

Bug #25367

closed

rsaauth does not encrypt new passwords entered in forgot password form

Added by Christian Clemens about 13 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Authentication
Target version:
-
Start date:
2011-03-21
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.5
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

I'm using felogin with saltedpasswords and rsaauth. Login works fine: I enter username and password and after clicking on loginbutton, password is modfied via javascript. Source code of login form shows:

... onsubmit="tx_rsaauth_feencrypt(this);; return true;" ...

In the form in which user can change his password (form shown, after clicking on link in forgot-password-mail), there seems to be no modification via JavaScript. After clicking the button to save the new password, password field is not modified, in source code of the form I could not find a any javascript like the one above.

Can anybody reproduce this problem?

I could send a link and a username per e-mail if needed.

(issue imported from #M18002)


Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Task #80018: Deprecate usage of EXT:rsaauthClosed2017-07-13

Actions
Related to TYPO3 Core - Task #81852: Deprecate EXT:rsaauthClosedGeorg Ringer2017-07-13

Actions
Has duplicate TYPO3 Core - Bug #44160: rsaauth doesn't secure the change password form of feloginClosed2012-12-20

Actions
Actions #1

Updated by Felix Nagel almost 13 years ago

  • Target version deleted (0)

I am able to reproduce this issue in TYPO3 4.4.7

Actions #2

Updated by Felix Nagel almost 13 years ago

Seems the password change does work, but the pass is submitted in plain, not encryped. So this is a security issue, not a show stopper.

Actions #3

Updated by Steffen Gebert over 12 years ago

  • Category changed from Communication to Authentication
Actions #4

Updated by Gerrit Code Review over 10 years ago

  • Status changed from New to Under Review

Patch set 17 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17256

Actions #5

Updated by Gerrit Code Review about 9 years ago

Patch set 18 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/17256

Actions #6

Updated by Christian Kuhn almost 9 years ago

  • Status changed from Under Review to New
  • Is Regression set to No

The patch was stalled in review for a very long time. It was abandoned to keep the review list a bit more clean. If anyone wants to pick up again, feel free to push a new patch.

Actions #7

Updated by Riccardo De Contardi about 8 years ago

I think it is still relevant in current 8.1 master: if I put a login form in a page and disable javascript, then it is not possible to log in,

Actions #8

Updated by Riccardo De Contardi over 6 years ago

  • Related to Task #80018: Deprecate usage of EXT:rsaauth added
Actions #9

Updated by Riccardo De Contardi over 6 years ago

Actions #10

Updated by Georg Ringer over 5 years ago

  • Status changed from New to Closed

I am closing this issue even though it is still valid. Reasion is that rsauth has been deprecated and will be removed in 10. HTTPS is, especially with let's encrypt, a must have and easy to get. also from SEO perspective it is now important and if https is available rsaauth can be dropped also in 7 or 8.

Actions

Also available in: Atom PDF