Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DRY medias header location + Fix repost on reload page #718

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sudwebdesign
Copy link
Contributor

$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

$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
```
@bazooka07
Copy link
Collaborator

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

@bazooka07
Copy link
Collaborator

Voir PR ##726 pour redirection après traitement de $_POST

@sudwebdesign
Copy link
Contributor Author

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 👍 ) > F5 + Ok == invalid or expired token

@bazooka07
Copy link
Collaborator

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

@sudwebdesign
Copy link
Contributor Author

sudwebdesign commented Jan 31, 2024

Merci pour l'info, ça peut-être utile 👍

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
Copy link
Contributor Author

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 ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants