Anomalie #25110
ferméEdition OnlyOffice en erreur à partir du tag 2301.0.4 : URL non valide
Description
En tant que utilisateur, quand j'essaie d'éditer un document (OnlyOffice, Collabora), j'ai me message d'erreur en PJ.
Anomalie reproduite sur la demo.
Non reproduite sur les tags 2301.0.3 et inférieurs.
Fichiers
Mis à jour par Charles BATIONO il y a plus d'un an
- Priorité changé de 2-Sérieux à 0-Bloquant
L'anomalie est liée aux lignes de code suivantes (ligne 53-57 du fichier DocumentEditorController.php ):
if (!Validator::notEmpty()->ip()->validate($args['uri'] ?? null) && !Validator::notEmpty()->url()->validate($args['uri'] ?? null)) {
return ['errors' => "Editor 'uri' is not a valid URL or IP address", 'lang' => 'editorHasNoValidUrlOrIp'];
} elseif (!preg_match('/^(https?:\/\/)?([\da-z.-]+)\.([a-z.]{2,6})([\/\w .-]*)*\/?$|^(https?:\/\/)?((25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)$/', $args['uri'] ?? null)) {
return ['errors' => "Editor 'uri' is not a valid URL or IP address format", 'lang' => 'editorHasNoValidUrlOrIp'];
}
En commentant ces lignes, on n'a plus le message d'erreur.
Mis à jour par Emmanuel DILLARD il y a plus d'un an
- Sujet changé de Edition OnlyOffice KO Tag 2301.0.4 (probable sur la 2301.1.0) à Edition OnlyOffice en erreur à partir du tag 2301.0.4 : URL non valide
Mis à jour par Emmanuel DILLARD il y a plus d'un an
- Statut changé de A qualifier à R&D - En cours
- Assigné à changé de Emmanuel DILLARD à Nicolas LE BOZEC
- Version source mis à 2301.1
Mis à jour par Emmanuel DILLARD il y a plus d'un an
- Lié à Anomalie #24124: CVE - Sécurité - Filtrage Paramétrage URI serveurs éditeurs ajouté
Mis à jour par Emmanuel DILLARD il y a plus d'un an
- Version source changé de 2301.1 à 2301
Mis à jour par Hamza HRAMCHI il y a plus d'un an
- Assigné à changé de Nicolas LE BOZEC à Hamza HRAMCHI
Mis à jour par GIT LAB il y a plus d'un an
[CREATION] MR sur 2301_releases (fix/25110/2301) par Alex ORLUC alex.orluc@maarch.org
https://labs.maarch.org/maarch/MaarchCourrier/commit/7bdb3ab81ad96f1da8aca38270e1d395f83696d0
Mis à jour par Hamza HRAMCHI il y a plus d'un an
- Statut changé de R&D - En cours à R&D - En test
Mis à jour par Hamza HRAMCHI il y a plus d'un an
- Statut changé de R&D - En test à R&D - Terminé
- Assigné à changé de Hamza HRAMCHI à Nicolas LE BOZEC
Mis à jour par GIT LAB il y a plus d'un an
[CLOTURE] MR sur 2301_releases (fix/25110/2301) par Hamza HRAMCHI
https://labs.maarch.org/maarch/MaarchCourrier/-/merge_requests/1101
Mis à jour par GIT LAB il y a plus d'un an
[CREATION] MR sur 2301_releases (fix/25110/2301) par lebozec nicolas.lebozec@xelians.fr
https://labs.maarch.org/maarch/MaarchCourrier/commit/578ab19a410ece2e43d622db583b0ba1ebc4fea7
Mis à jour par GIT LAB il y a plus d'un an
[CLOTURE] MR sur 2301_releases (fix/25110/2301) par Alex ORLUC
https://labs.maarch.org/maarch/MaarchCourrier/-/merge_requests/1108