Bug #34568

rsaauth can not be used multiple times on one page

Added by Alexander Stehlik almost 8 years ago. Updated over 2 years ago.

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

100%

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

Description

When using rsaauth for the felogin extension it is not possible to have another login form on the same page that also uses rsaauth for encrypting the user/password data.

As far as I can see there are two possible solutions:

  1. the storage need to be able to store multiple key pairs (at the moment only one key pair can be stored)
  2. if a key pair is generated it needs to be stored in a static variable so that it can be used in other forms

Please let me know if I can help out with a patch for this.


Related issues

Related to TYPO3 Core - Bug #24877: rsaauth with more than one login form on one page doesn't work Closed 2011-01-28
Duplicated by Front End User Registration - Bug #38904: sr_feuser_register registration form not working if felogin placed on the same page Closed 2012-07-12
Duplicated by Front End User Registration - Bug #40602: Conflicts with fe_login Closed 2012-09-04
Duplicated by Front End User Registration - Bug #42050: felogin + sr_feuser_register on one page dont work Closed 2012-10-16
Duplicated by Front End User Registration - Bug #43379: FE loginSecurityLevel = rsa not working Closed 2012-11-28

Associated revisions

Revision 435f778c (diff)
Added by Nicole Cordes over 4 years ago

[FEATURE] Add api for rsaauth encryption

This patch introduces new API which eases the use of RSA encryption
across the core and for third party extensions.

For backend encryption a new RequireJS module is included which finds
fields to encrypt automatically by their data attribute.

For frontend encryption the JavaScript was rewritten to work the same
way the RequireJS module works but with plain (vanilla) JavaScript.

Releases: master
Resolves: #67932
Resolves: #34568
Change-Id: Id35dd65d1019a8c16858f03989ff48e3434698e4
Reviewed-on: http://review.typo3.org/40938
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>
Reviewed-by: Georg Ringer <>
Tested-by: Georg Ringer <>

History

#1 Updated by Christian Jul Jensen almost 8 years ago

  • Status changed from New to Needs Feedback
  • Assignee set to Steffen Ritter

Steffen, you are probably the right one to say if this is feasible. Or if there are other things to consider.

#2 Updated by Steffen Ritter almost 8 years ago

  • Status changed from Needs Feedback to Accepted
  • Assignee deleted (Steffen Ritter)

Yes, know this behaviour - I think in the second way would fit better...

#3 Updated by Online Now! GmbH almost 8 years ago

Please make also sure the javascript files for the encrypting will be added only one time.

#4 Updated by Andreas Kiessling almost 8 years ago

Since the next release of sr_feuser_register (3) will ship with support for salted_passwords and rsa encrypted passwords (and adds conflict status with kb_md5fepw), i think it will be pretty common to have rsa protected login form and the password fields on the same page.
Identifying this problem took quite some time and although there is (now) a note and a link in the sr_feuser_register documentation regarding this error, this should be fixed for the next releases IMHO.

#5 Updated by Andreas Kiessling almost 8 years ago

Related to #24877 which has patches pending in Gerrit https://review.typo3.org/#/c/10485/

#6 Updated by hwt3 no-lastname-given over 5 years ago

What is the current state of this Issue?

The same problem exists in TYPO3 5.2.x, e.g. with sr_feuser_register 3.1.x

#7 Updated by Nicole Cordes over 4 years ago

  • Assignee set to Nicole Cordes
  • Target version set to 7.4 (Backend)
  • Is Regression set to No

#8 Updated by Gerrit Code Review over 4 years ago

  • Status changed from Accepted to Under Review

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

#9 Updated by Gerrit Code Review over 4 years ago

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

#10 Updated by Gerrit Code Review over 4 years ago

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

#11 Updated by Gerrit Code Review over 4 years ago

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

#12 Updated by Gerrit Code Review over 4 years ago

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

#13 Updated by Gerrit Code Review over 4 years ago

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

#14 Updated by Gerrit Code Review over 4 years ago

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

#15 Updated by Gerrit Code Review over 4 years ago

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

#16 Updated by Gerrit Code Review over 4 years ago

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

#17 Updated by Gerrit Code Review over 4 years ago

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

#18 Updated by Gerrit Code Review over 4 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/40938

#19 Updated by Gerrit Code Review over 4 years ago

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

#20 Updated by Nicole Cordes over 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#21 Updated by Riccardo De Contardi over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF