developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 14 years ago

Closed 14 years ago

#304 closed bug (fixed)

The javascript function verifyForm doesn't implement URL validation

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

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 (1)

verifyurl.diff (855 bytes) - added by arnaudj 14 years ago.
Basic implementation

Download all attachments as: .zip

Change History (4)

comment:1 Changed 14 years ago by laurentj

  • Milestone changed from Jelix 1.0beta3.1 to Jelix 1.0
  • Priority changed from normal to low
  • Severity changed from normal to minor
  • Version changed from trunk to 1.0 beta3

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.

Changed 14 years ago by arnaudj

Basic implementation

comment:2 Changed 14 years ago 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.

comment:3 Changed 14 years ago by laurentj

  • Milestone changed from Jelix 1.0 to Jelix 1.0beta3.1
  • Resolution set to fixed
  • Status changed from new to closed

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

Note: See TracTickets for help on using tickets.