[TYPO3-core] RFC #11016: Implement RSA authentication for BE and FE

Dmitry Dulepov dmitry.dulepov at gmail.com
Sat May 2 19:41:56 CEST 2009


Hi!

Oliver Hader wrote:
> I just had a quick run over the files and it looks good so far. I
> discovered some minor things with naming and PHPdocs. Furthermore the
> factory methods seem to be strict as for example the storage factory
> implements a "singleton storage". However for this specific case it's
> okay...
> 
> What should I do about changes? Are the patches you posted here based on
> the Forge project? If they are, I could post the diff to the latest SVN
> revision.

Thanks for looking at it! May be you can tell what you propose to change and we discuss it here? I am ok to make changes but I may need to explain why I did some things one way and not the other :)

For example, I used factories because I like this design pattern. I used it for ages and it allows to hide the selection logic inside one place.

-- 
Dmitry Dulepov
In TYPO3 blog: http://dmitry-dulepov.com/article/when-do-you-need-chash-in-typo3.html
LinkedIn: http://www.linkedin.com/in/dmitrydulepov
Twitter: http://twitter.com/dmitryd
Skype: liels_bugs


More information about the TYPO3-team-core mailing list