PluXml icon indicating copy to clipboard operation
PluXml copied to clipboard

DRY medias header location + Fix repost on reload page

Open sudwebdesign opened this issue 2 years ago • 6 comments

$redirect & $posted for AdminMediasPrepend hook

Fix when move on folder & reload page after browser say : You want repost data? If click Ok, PluXml say :

Security error : invalid or expired token

sudwebdesign avatar Jan 25 '24 03:01 sudwebdesign

PluXml does not say "Security error : invalid or expired token" voir PR #724

bazooka07 avatar Jan 28 '24 09:01 bazooka07

Voir PR #https://github.com/pluxml/PluXml/pull/726 pour redirection après traitement de $_POST

bazooka07 avatar Jan 28 '24 10:01 bazooka07

PluXml does not say "Security error : invalid or expired token" voir PR #724

Si avec F5² (qui fait ça appart les devs)

Pour reproduire ; on choisit un dossier > Ok > F5 + Ok (yep :+1: ) > F5 + Ok == invalid or expired token

sudwebdesign avatar Jan 28 '24 21:01 sudwebdesign

public static function validateFormToken($request='') {
    if($_SERVER['REQUEST_METHOD']=='POST' AND isset($_SESSION['formtoken'])) {
        # .....

        if(!empty($_SESSION['formtoken'])) {
            while(count($_SESSION['formtoken']) > 5) {
                $oldTime = array_shift($_SESSION['formtoken']);
            }
        }
    }
}

Maintenant tu as droit à 5 tentatives

bazooka07 avatar Jan 29 '24 07:01 bazooka07

Merci pour l'info, ça peut-être utile :+1:

Imaginons un cas d'utilisation :

Un utilisateur attend d'un autre qu'il envoie des images... Il choisit le dossier là ok, il recharge un peu après avec F5 ou avec l’icône du navi (déjà il se demande oui ou non) bon il dit ok et aucune erreur de token (maintenant, avant oui).

Et il recharge la page encore une fois de plus, (il a plus peur, ça passe). Bin là une erreur (normale) apparaît et il est bloqué.

Sauf que le F5 suivant (avec re-postage) passe bien sans l'erreur, le suivant non et suivant oui et ainsi de suite, Est-ce normal, ou pas?

[edit] C'est bien que ça passe, car ma permis d'évité de perdre des données après une longue pause. (se connecte avec un autre onglet) re-poste et ouf c'est sauvé.

sudwebdesign avatar Jan 31 '24 16:01 sudwebdesign

De plus ça permet aux codeurs de désactiver la redirection avec une seule ligne a commenter, quelques fois ça ma aidé à trouver ou ça cloche, la redir cache parfois des choses ;)

sudwebdesign avatar Jan 31 '24 16:01 sudwebdesign