Ticket #445 (closed enhancement: fixed)

Opened 10 months ago

Last modified 8 months ago

Désactiver les sessions pour cmdline.php

Reported by: hugues Assigned to: Julien
Priority: low Milestone: Jelix 1.0.3
Component: jelix-scripts Version: 1.0.1
Severity: trivial Keywords:
Cc: Php version:
Review: review+ Hosting Provider:
Documentation needed: 0 Blocking:

Description

Ça serait top de désactiver la session pour les scripts qui tournent via cmdline.php

Attachments

445-jSession-no-session-for-cmdline-requests.diff (0.7 kB) - added by Julien on 03/16/08 03:22:47.

Change History

01/30/08 21:10:08 changed by laurentj

  • summary changed from désactivé session pour cmdline.php to Désactiver lesøsessions pour cmdline.php.

02/01/08 10:26:19 changed by laurentj

  • summary changed from Désactiver lesøsessions pour cmdline.php to Désactiver les sessions pour cmdline.php.

02/26/08 16:02:36 changed by laurentj

  • priority changed from normal to low.
  • review changed.
  • severity changed from normal to trivial.

03/16/08 02:59:19 changed by Julien

  • owner set to Julien.
  • docneeded changed.

03/16/08 02:59:29 changed by Julien

  • status changed from new to assigned.

03/16/08 03:22:47 changed by Julien

  • attachment 445-jSession-no-session-for-cmdline-requests.diff added.

03/16/08 03:24:53 changed by Julien

  • review set to review?.

Voici le patch.

J'ai fait au plus simple, à savoir tester le type de requête auquel on répond.

03/16/08 15:34:03 changed by laurentj

Ça ne pose pas de problème le session_write_close(); si il n'y a pas eu le session_start() ?

03/16/08 16:59:48 changed by Julien

D'après les tests pratiques non, pas de problème.

L'idée pourrait être de rajouter une propriété : 

 protected $_started = false;

que l'on passerait à true lors du session_start effectif.

Du coup, jSession::end() vérifierai cet état pour savoir s'il faut faire le session_write_close().

Re-patch dans ce sens ?

03/16/08 21:33:53 changed by laurentj

  • review changed from review? to review+.
  • milestone set to Jelix 1.0.3.

laissons comme ça, on verra si on a des remontées de bug.

Patch à appliquer dans la branche et le trunk.

03/20/08 01:49:17 changed by Julien

  • status changed from assigned to closed.
  • resolution set to fixed.

Ok, patch intégré.

Download in other formats: Comma-delimited Text Tab-delimited Text RSS Feed