Projet

Général

Profil

Actions

Anomalie #2005

fermé

Prévenir l'envoi de requêtes SQL invalide (type NULL, etc.)

Ajouté par Jérôme Augé il y a plus de 14 ans. Mis à jour il y a environ 14 ans.

Statut:
Intégré
Priorité:
Normal
Assigné à:
Version cible:
Début:
26/09/2011
Echéance:
% réalisé:

100%

Temps estimé:
9:00 h
Version source:
Solution proposée:

Ajouter un paramètre CORE pour mettre in mode sqlstrict. Ce paramètre permet de retourner une exception si une requête est invalide.
Activation du mode sql strict :
- mise à jour platform : désactivé
- installation platform : activé

Si le mode strict n'est pas actif une erreur dans les log est quand même affichée. (voir #649)

Principaux fichiers impactés:

Class.DbObj.php (exec_query)
Lib.Common.php (simpleQuery)

Complexité:
Moyen
Contrôle:

TAUFU : Suite Database
Class ObjectQuery
Test exec_query avec paramètre activé et non activé

Class SimpleQuery
Test fonction simplequery avec paramètre strict activé et non activé

Thème:
Régression:

Description

Des requêtes SQL invalides peuvent être générés du style :
  • ERROR: null value in column "vaultid" violates not-null constraint: insert into docvaultindex(docid,vaultid) values (E'4687',null)
  • ERROR: invalid input syntax for integer: "": select * from vaultdiskstorage where vaultdiskstorage.id_file=E''

Si on sait que "vaultid", ou "id_file", ne peuvent pas être NULL, peut-être pourrait-on vérifier s'ils sont valide avant d'envoyer la requête à PostgreSQL ? et de remonter l'erreur à l'appelant ?


Demandes liées 2 (1 ouverte1 fermée)

Lié à Core - Anomalie #649: Afficher la stacktrace lors d'une erreur exec_queryNouveau22/03/2010

Actions
Lié à Core - Anomalie #1739: [Vault] Erreur lors de la première insertion d'un fichier dans le vaultRésoluÉric Brison

Actions

Mis à jour par Marc Claverie il y a plus de 14 ans

  • Statut changé de Nouveau à À analyser
  • Assigné à mis à Éric Brison

Mis à jour par Éric Brison il y a plus de 14 ans

  • Version cible mis à 3.1.2

Mis à jour par Éric Brison il y a plus de 14 ans

  • Temps estimé mis à 9:00 h
  • Solution proposée mis à jour (diff)
  • Principaux fichiers impactés mis à jour (diff)
  • Complexité mis à Moyen
  • Contrôle mis à jour (diff)

Mis à jour par Marc Claverie il y a plus de 14 ans

  • Solution proposée mis à jour (diff)

Mis à jour par Marc Claverie il y a plus de 14 ans

  • Version cible changé de 3.2 rx à 3.2 sprint 3

Mis à jour par Éric Brison il y a plus de 14 ans

  • Assigné à Éric Brison supprimé

Mis à jour par Éric Brison il y a environ 14 ans

  • Statut changé de À analyser à Analysé

Mis à jour par Éric Brison il y a environ 14 ans

  • Assigné à mis à Éric Brison

Mis à jour par Éric Brison il y a environ 14 ans

  • Statut changé de Analysé à Intégré
  • % réalisé changé de 0 à 100

Appliqué par commit commit:063215fcd6c34924760179caee2149a7a8ed796e.

Actions

Formats disponibles : Atom PDF