Feature #67932

EXT:rsaauth should provide a generic interface

Added by Philipp Kerling over 4 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2015-07-06
Due date:
% Done:

100%

PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

The rsaauth extension provides functions for encrypting via RSA in JavaScript on the web page and the corresponding decrypt functions in PHP. It also has integration into the backend forms and frontend login forms for encrypting login passwords when using EXT:felogin via hooks.

What I think is missing is a more general interface for using the RSA functionality in custom extensions, e.g. login forms and profile edit forms. Only EXT:felogin works out of the box since hooks are provided for that, but I don't see why other extensions shouldn't be able to use RSA authentication/encryption explicitly. It's possible to use the felogin hooks in custom login forms to get identical behaviour, but frontend forms for allowing an user to change his/her password are very difficult to implement (see also https://github.com/pagemachine/hairu/issues/4 and https://forge.typo3.org/issues/25367).


Related issues

Related to TYPO3 Core - Bug #69007: Change 67932 introduced a breaking change in felogin but rst was forgotten Closed 2015-08-13

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 Kuhn over 4 years ago

Hey. Your analysis is perfectly right, the current rsaauth architecture misses a generic interface to handle any data, the current hook architecture is doomed. We thought about splitting the extension into two clean parts, but nobody got their fingers dirty, yet.

#2 Updated by Gerrit Code Review over 4 years ago

  • Status changed from New to Under Review

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

#3 Updated by Gerrit Code Review over 4 years ago

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

#4 Updated by Gerrit Code Review over 4 years ago

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

#5 Updated by Gerrit Code Review over 4 years ago

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

#6 Updated by Gerrit Code Review over 4 years ago

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

#7 Updated by Gerrit Code Review over 4 years ago

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

#8 Updated by Gerrit Code Review over 4 years ago

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 Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF