Ticket #304 (closed bug: fixed)

Opened 1 year ago

Last modified 1 year ago

The javascript function verifyForm doesn't implement URL validation

Reported by: arnaudj Assigned to:
Priority: low Milestone: Jelix 1.0beta3.1
Component: jelix:forms Version: 1.0 beta3
Severity: minor Keywords:
Cc: Php version:
Review: Hosting Provider:
Documentation needed: Blocking:

Description

In jForm, the URL validation seems not to be implemented by the javascript function verifyForm. Cf http://developer.jelix.org/browser/trunk/lib/jelix-www/js/jforms.js#L183

Attachments

verifyurl.diff (0.8 kB) - added by arnaudj on 10/15/07 15:58:39.
Basic implementation

Change History

10/15/07 09:55:44 changed by laurentj

  • priority changed from normal to low.
  • version changed from trunk to 1.0 beta3.
  • severity changed from normal to minor.
  • milestone changed from Jelix 1.0beta3.1 to Jelix 1.0.

The bug will be attached for the beta3.1 milestone only if a patch is provided (Beta3.1 will be released this week). I don't have time to fix it.

10/15/07 15:58:39 changed by arnaudj

  • attachment verifyurl.diff added.

Basic implementation

10/15/07 16:11:57 changed by arnaudj

Dans le patch fourni :

  • Les URL acceptées sont de la forme : protocole + nom de domaine ou IP + / n'importe quoi.
  • Les protocoles acceptés sont : http://, https://, ftp://, ftps://

Exemples d'URL acceptées :

Exemples d'URL rejetées

Par contre, je n'ai pas vérifié la cohérence entre ces résultats et ceux de jFilter.

10/16/07 01:04:00 changed by laurentj

  • status changed from new to closed.
  • resolution set to fixed.
  • milestone changed from Jelix 1.0 to Jelix 1.0beta3.1.

Merci pour le patch. Par contre ton expression régulière n'est pas valide. En effet, les protocoles d'internet ne se limitent pas à http et ftp ;-). De plus il y a une mauvaise vérification quand on donne une adresse IP.

Toutefois j'ai finalement eu le temps de corriger le bug, vu qu'il est très semblable au ticket #297. Corrigé dans le trunk et dans la branche beta3.x

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