Question

在我的应用程序中，一旦用户登录，便会转到主页，在其中可以查看其详细信息。有一个“编辑配置文件”按钮，在该按钮中，用户将被带到可以编辑数据的页面。一旦编辑成功，他将被重定向回主页。但是在这里，它被重定向到登录页面。我认为该会话意外过期。如何克服这个问题？

//这是我的更新信息控制器

/**
 * @Route("/update/{id}", name="update")
 * @param $id
 * @param Request $request
 * @param UserPasswordEncoderInterface $passwordEncoder
 * @param UserInterface $loggedUser
 * @return \Symfony\Component\HttpFoundation\RedirectResponse|Response
 */

public function updateUser($id,Request $request, UserPasswordEncoderInterface $passwordEncoder, UrlGeneratorInterface $urlGenerator){

    $loggedUser = $this->get('security.token_storage')->getToken()->getUser()->getId();

    if ($id == $loggedUser){
        $em = $this->getDoctrine()->getManager();
        $conn =$em->getConnection();
        $user = $em->find(User::class,$id);

        $form = $this->createForm(RegisterType::class,$user, [
            'validation_groups' => ['update'],
        ]);


        $form->handleRequest($request);

        if($form->isSubmitted() && $form->isValid()) {

            $file = $request->files->get('register')['image'];
            if($file){
                $fileName = md5(uniqid()).'.'.$file->guessExtension();
                $file->move(
                    $this->getParameter('uploads_dir'), $fileName
                );
                $user->setImage($fileName);
            }

            if($user->getPassword() !="") {
                $user->setPassword($passwordEncoder->encodePassword($user,$user->getPassword()));

                $sql = '
                    UPDATE user
                    SET first_name = :firstName, last_name = :lastName, id_number = :idNumber, phone_number = :phoneNumber, address = :address, password = :password
                    WHERE id= :id
    ';
                $stmt = $conn->prepare($sql);
                $stmt->execute(['firstName' => $user->getFirstName(),
                    'lastName' => $user->getLastName(),
                    'idNumber' => $user->getIdNumber(),
                    'phoneNumber' => $user->getPhoneNumber(),
                    'address' => $user->getAddress(),
                    'password' => $user->getPassword(),
                    'id' => $id]);
            } else {
                $sql = '
                    UPDATE user
                    SET first_name = :firstName, last_name = :lastName, id_number = :idNumber, phone_number = :phoneNumber, address = :address
                    WHERE id= :id
    ';
                $stmt = $conn->prepare($sql);
                $stmt->execute(['firstName' => $user->getFirstName(),
                    'lastName' => $user->getLastName(),
                    'idNumber' => $user->getIdNumber(),
                    'phoneNumber' => $user->getPhoneNumber(),
                    'address' => $user->getAddress(),
                    'id' => $id]);
            }



            return new RedirectResponse($urlGenerator->generate('home'));
        }
    } else {
        return new RedirectResponse($urlGenerator->generate('home'));
    }

    return $this->render('register/update.html.twig', [
        'form'=>$form->createView(),
    ]);
}

//这是RegisterType形式

class RegisterType extends AbstractType
{
    public function buildForm(FormBuilderInterface $builder, array $options)
    {
        $builder
            ->add('email',EmailType::class,[
                'label'=>'Email',
                'required' => false,
                'attr'=>['placeholder'=>"Email"]
            ])
            ->add('password',RepeatedType::class,[
                'type' => PasswordType::class,
                'invalid_message' => 'The password fields must match.',
                'required' => false,
                'options' => ['attr' => ['class' => 'password-field']],
                'first_options'  => ['label' => 'Password','attr'=>['placeholder'=>"Password"]],
                'second_options' => ['label' => 'Confirm Password','attr'=>['placeholder'=>"Confirm Password"]],
            ])
            ->add('firstName',TextType::class,['label'=>'First Name',  'attr'=>['placeholder'=>"First Name"]])
            ->add('lastName',TextType::class,['label'=>'Last Name','attr'=>['placeholder'=>"Last Name"]])
            ->add('address',TextareaType::class,['required' => false,'label'=>'Address','attr'=>['placeholder'=>"Address"]])
            ->add('idNumber',TextType::class,['label'=>'NIC Number','attr'=>['placeholder'=>"NIC Number"]])
            ->add('phoneNumber',TelType::class,['label'=>'Phone Number','attr'=>['placeholder'=>"Phone Number"]])
            ->add('image',FileType::class,['label'=>'Photo','required'=>false,'attr'=>['hidden'=>"hidden", 'accept'=>"image/jpeg, image/png"]])
            ->add('save',SubmitType::class,[
                'label'=>'Register',
                'attr' => [
                    'class'=>"btn btn-outline-success float-right"
                ]
            ])

        ;
    }

    public function configureOptions(OptionsResolver $resolver)
    {
        $resolver->setDefaults([
            'data_class' => User::class,
        ]);
    }
}

//这是我的用户类别

class User implements UserInterface{

/**
 * @ORM\Id()
 * @ORM\GeneratedValue()
 * @ORM\Column(type="integer")
 */
private $id;

/**
     * @ORM\Column(type="string", length=180, unique=true)
 * @Assert\Email()
 * @Assert\NotBlank()
 */
private $email;

/**
 * @ORM\Column(type="json")
 */
private $roles = [];

/**
 * @var string The hashed password
 * @ORM\Column(type="string")
 * @Assert\NotBlank()
 */
private $password;

/**
 * @ORM\Column(type="string",length=255)
 * @Assert\NotBlank(groups={"update"})
 *
 */
private $firstName;

/**
 * @ORM\Column(type="string",length=255)
 * @Assert\NotBlank(groups={"update"})
 */
private $lastName;

/**
 * @ORM\Column(type="string",length=255,nullable=true)
 *
 */
private $image;

/**
 * @ORM\Column(type="string", nullable=true)
 */
private $address;

/**
 * @ORM\Column(type="string",length=10)
 * @Assert\Length("10",groups={"update"})
 */
private $phoneNumber;

/**
 * @ORM\Column(type="string",length=10)
 * @Assert\NotBlank(groups={"update"})
 * @Assert\Length("10",groups={"update"})
 */
private $idNumber;

/**
 * @ORM\OneToMany(targetEntity="App\Entity\Vehicle", mappedBy="user")
 */
private $vehicle;

/**
 * @ORM\OneToOne(targetEntity="App\Entity\Account", inversedBy="user")
 */
private $account;


public function __construct()
{
    $this->vehicle = new ArrayCollection();
}

public function getId(): ?int
{
    return $this->id;
}

public function getEmail()
{
    return $this->email;
}

public function setEmail( $email): self
{
    $this->email = $email;

    return $this;
}

/**
 * A visual identifier that represents this user.
 *
 * @see UserInterface
 */
public function getUsername()
{
    return (string) $this->email;
}

public function getRoles(): ?array
{
    return $this->roles;
}

public function setRoles(array $roles): self
{
    $this->roles = $roles;

    return $this;
}

public function getPassword()
{
    return $this->password;
}

public function setPassword($password): self
{
    $this->password = $password;

    return $this;
}

/**
 * @see UserInterface
 */
public function getSalt()
{
    // not needed when using the "bcrypt" algorithm in security.yaml
}

/**
 * @see UserInterface
 */
public function eraseCredentials()
{
    // If you store any temporary, sensitive data on the user, clear it here
    // $this->plainPassword = null;
}

public function getFirstName()
{
    return $this->firstName;
}

public function setFirstName( $firstName): self
{
    $this->firstName = $firstName;

    return $this;
}

public function getLastName()
{
    return $this->lastName;
}

public function setLastName( $lastName): self
{
    $this->lastName = $lastName;

    return $this;
}

public function getImage(): ?string
{
    return $this->image;
}

public function setImage(string $image): self
{
    $this->image = $image;

    return $this;
}

public function getAddress()
{
    return $this->address;
}

public function setAddress( $address): self
{
    $this->address = $address;

    return $this;
}

public function getIdNumber()
{
    return $this->idNumber;
}

public function setIdNumber( $idNumber): self
{
    $this->idNumber = $idNumber;

    return $this;
}

public function getPhoneNumber()
{
    return $this->phoneNumber;
}

public function setPhoneNumber( $phoneNumber): self
{
    $this->phoneNumber = $phoneNumber;

    return $this;
}

/**
 * @return Collection|Vehicle[]
 */
public function getVehicle(): Collection
{
    return $this->vehicle;
}

public function addVehicle(Vehicle $vehicle): self
{
    if (!$this->vehicle->contains($vehicle)) {
        $this->vehicle[] = $vehicle;
        $vehicle->setUser($this);
    }

    return $this;
}

public function removeVehicle(Vehicle $vehicle): self
{
    if ($this->vehicle->contains($vehicle)) {
        $this->vehicle->removeElement($vehicle);
        // set the owning side to null (unless already changed)
        if ($vehicle->getUser() === $this) {
            $vehicle->setUser(null);
        }
    }

    return $this;
}

public function getAccount(): ?Account
{
    return $this->account;
}

public function setAccount(?Account $account): self
{
    $this->account = $account;

    return $this;
}

}

Answer 1

假设您使用的是默认投票者和实体安全用户提供程序。

这应该适用于Symfony 3.4+，但是知道您使用的是哪个版本的Symfony，将可以采用其他方法。

在每个请求结束时（除非您的防火墙是无状态的），您的   用户对象被序列化到会话。在下一个开始   请求，将其反序列化，然后传递给您的用户提供商   “刷新”它（例如，教义查询新用户）。

然后是两个User对象（会话中的原始对象和   已刷新的用户对象）进行“比较”，以查看它们是否“相等”。通过   默认情况下，核心AbstractToken类比较以下项的返回值：   getPassword（），getSalt（）和getUsername（）方法。如果有   这些是不同的，您的用户将被注销。这是一种安全   采取措施以确保恶意用户可以在以下情况下取消身份验证：   核心用户数据更改。

但是，在某些情况下，此过程可能会导致意外   身份验证问题。如果您在验证时遇到问题，   可能是您认证成功，但是您立即   第一次重定向后失去身份验证。

来源：https://symfony.com/doc/current/security/user_provider.html#understanding-how-users-are-refreshed-from-the-session

问题似乎是由

引起的

$user->setPassword($passwordEncoder->encodePassword($user,$user->getPassword()));

这将从提交的密码中生成一个新的已散列密码，即使用户状态相同，也会使用户状态无效。

您将需要存储用户的纯文本密码，验证它是否已更改，并且仅在密码更改后才应用它。

另外，您的image表单设置无效，因为您的User::$image需要一个字符串，但是表单将上传File对象（导致无效的实体状态或调用{{1 }}并更改图像）。您应该为图像上传使用单独的属性，并在视图中手动绘制当前图像，或者考虑在Form中而不是在控制器中使用数据转换器来处理状态更改。参见：https://symfony.com/doc/current/form/data_transformers.html

用File::__toString和password字段替换当前的image和plainPassword表单字段。

uploadImage

您还应该认真考虑使用DTO而不是Doctrine的直接用户实体来管理数据，以防止无效的实体状态。

然后在用户实体中创建属性和getter / setter方法，以存储表单值。

class RegisterType extends AbstractType
{
    public function buildForm(FormBuilderInterface $builder, array $options)
    {
        $builder
            //...
            ->add('plainPassword',RepeatedType::class,[
                'type' => PasswordType::class,
                'invalid_message' => 'The password fields must match.',
                'required' => false,
                'options' => ['attr' => ['class' => 'password-field']],
                'first_options'  => ['label' => 'Password','attr'=>['placeholder'=>"Password"]],
                'second_options' => ['label' => 'Confirm Password','attr'=>['placeholder'=>"Confirm Password"]],
            ])
            ->add('uploadImage',FileType::class,['label'=>'Photo','required'=>false,'attr'=>['hidden'=>"hidden", 'accept'=>"image/jpeg, image/png"]]);
      //...
}

由于使用的是实体管理器和class User implements UserInterface { /** * @var string */ private $plainPassword = ''; /** * @var File|null */ private $uploadImage; public function getPlainPassword(): string { return $this->plainPassword; } public function setPlainPassword(string $plainPassword): void { $this->plainPassword = $plainPassword; } /** * @see UserInterface */ public function eraseCredentials() { $this->plainPassword = null; } public function getUploadImage(): ?File { return $this->uploadImage; } public function setUploadImage(?File $file): void { $this->uploadImage = $file; } //... }字段，因此可以删除手动更新查询。由于RegisterType会将更改直接应用到User对象。我还建议使用Paramconverter来受益于User对象的实体依赖注入。

$form->handleRequest()

如果您正在使用Symfony <4.1，则需要实现/** * @Route("/{user}/update", name="update", requirements={ "user":"\d+" }, methods={"GET","POST"}) * @param User $user * @param Request $request * @param UserPasswordEncoderInterface $passwordEncoder * @param UserInterface $loggedUser * @return Response */ public function updateUser(User $user, Request $request, UserPasswordEncoderInterface $passwordEncoder, UrlGeneratorInterface $urlGenerator): Response { $loggedinUser = $this->getUser(); //helper from ControllerTrait if ($loggedinUser && loggedinUser->getId() === $user->getId()) { $form = $this->createForm(RegisterType::class,$user, [ 'validation_groups' => ['update'], ]); $currentImage = $user->getImage(); $form->handleRequest($request); if ($form->isSubmitted() && $form->isValid()) { if ($file = $user->getUploadImage()) { //this logic should be moved to the Form using a data transformer $fileName = md5(uniqid()).'.'.$file->guessExtension(); $file->move( $this->getParameter('uploads_dir'), $fileName ); $user->setImage($fileName); } if ('' !== $user->getPlainPassword() && !$passwordEncoder->isPasswordValid($user->getPassword(), $user->getPlainPassword())) { //change password only when changed $user->setPassword($passwordEncoder->encodePassword($user, $user->getPlainPassword())); $user->eraseCredentials(); } $em = $this->getDoctrine()->getManager(); $em->flush(); return new RedirectResponse($urlGenerator->generate('home')); } return $this->render('register/update.html.twig', [ 'form'=>$form->createView(), ]); } return new RedirectResponse($urlGenerator->generate('home')); }并将\Serializable和serialize方法添加到User类，否则整个User对象将被序列化并更改无效。

unserialize

Answer 2

使用一个临时字段以预编码/预散列的形式保存纯文本密码（请参阅：https://symfony.com/doc/4.0/doctrine/registration_form.html#registration-password-max-该字段称为plainPassword或类似名称）。

我怀疑设置空密码时会发生一些意外行为，这可能会使会话缓存无效（symfony存储一些用户数据来确定是否必须从数据库中重新加载用户，并且如果相关数据已更改，则可能会记录用户出来）。重定向本身绝对不应该注销用户。

希望这足够了。

重定向到页面时，会话过期

2 个答案: