Skip to content

Commit

Permalink
Deprecate Pimcore\Tool\Authentication::authenticatePlaintext()
Browse files Browse the repository at this point in the history
  • Loading branch information
jdreesen authored and brusch committed Feb 17, 2023
1 parent a16e65e commit 39e2026
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@
- [Annotations] Using Annotations `@ResponseHeader` & `@ParamConverter`, `@Template` and rest from [SensioFrameworkExtraBundle](https://symfony.com/bundles/SensioFrameworkExtraBundle/current/index.html#annotations-for-controllers) is deprecated and will not be supported on Pimcore 11. Use `#[ResponseHeader]`,`#[DataObjectParam]` argument, `#[Template]` and other attributes instead.
- [Asset] Deprecated VR Preview. For details please see [#14111](https://github.com/pimcore/pimcore/issues/14111).
- [Authentication] The method `Pimcore\Tool\Authentication::authenticateHttpBasic()` has been deprecated and will be removed in Pimcore 11.
- [Authentication] The method `Pimcore\Tool\Authentication::authenticatePlaintext()` has been deprecated and will be removed in Pimcore 11.

## 10.5.13
- [Web2Print] Print document twig expressions are now executed in a sandbox with restrictive security policies (just like Sending mails and Dataobject Text Layouts introduced in 10.5.9).
Expand Down
8 changes: 8 additions & 0 deletions lib/Tool/Authentication.php
Original file line number Diff line number Diff line change
Expand Up @@ -25,13 +25,21 @@
class Authentication
{
/**
* @deprecated
*
* @param string $username
* @param string $password
*
* @return null|User
*/
public static function authenticatePlaintext($username, $password)
{
trigger_deprecation(
'pimcore/pimcore',
'10.6',
sprintf('%s is deprecated and will be removed in Pimcore 11', __METHOD__),
);

/** @var User $user */
$user = User::getByName($username);

Expand Down

0 comments on commit 39e2026

Please sign in to comment.