Projet

Général

Profil

Bug #31384

valider l'URL lors de création de service

Ajouté par Christophe Siraut il y a environ 5 ans. Mis à jour il y a environ 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Christophe Siraut
Catégorie:
-
Version cible:
-
Début:
13 mars 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Quand je tente de créer un service via hobo (par exemple welco ou fargo) en omettant le préfixe https:// dans l'URL du service, une déploiement est lancé, mais les fichiers se retrouvent créés dans /var/lib/$app/ au lieu de /var/lib/$app/$tenant.

Révisions associées

Révision e2ffab60 (diff)
Ajouté par Christophe Siraut il y a environ 5 ans

ServiceBase: validate base_url (#31384)

Historique

#3

Mis à jour par Christophe Siraut il y a environ 5 ans

  • Assigné à mis à Christophe Siraut
#4

Mis à jour par Christophe Siraut il y a environ 5 ans

  • Statut changé de Nouveau à Solution proposée
#5

Mis à jour par Christophe Siraut il y a environ 5 ans

  • Sujet changé de valider/compléter l'URL lors de création de service à valider l'URL lors de création de service
#6

Mis à jour par Nicolas Roche il y a environ 5 ans

  • Statut changé de Solution proposée à Solution validée

Peut-être réitérer plusieurs fois le même test (par exemple tester l’omission de 'http://' qui est l'objet du ticket)

 def test_invalid_url():
    kwargs = {'title': 'Combo test',
              'last_operational_success_timestamp': timezone.now(),
              'last_operational_check_timestamp': timezone.now(),
              'secret_key': '1nesüper5Cr!eteKAaY~'}
    cpt = 0; # slugs must be unique

    # accept well formated url
    for url in ('https://example.com', 'http://example.com'):
        cpt += 1
        slug = 'wesh' + str(cpt)
        combo = Combo(base_url=url, slug=slug ,**kwargs)
        combo.full_clean()
        combo.save()
        assert True  # no exception raised

    # reject baddly formated url
    for url in ('example.com', 'http:/example.com', 'file:///home/me'):
        cpt += 1
        slug = 'wesh' + str(cpt)
        with pytest.raises(ValidationError):
            combo = Combo(base_url=url, slug='wesh_'+str(cpt), **kwargs)
            combo.full_clean()
            combo.save()

A part ça, pour moi c'est bon.

#7

Mis à jour par Christophe Siraut il y a environ 5 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 9f715b356a1fc54f373adf732e9ba1d48eacf1b9 (HEAD -> master, origin/master, origin/HEAD, wip/31384-valider-completer-l-URL-lors-de-creation-de-service)
Author: Christophe Siraut <csiraut@entrouvert.com>
Date:   Fri Apr 12 14:46:50 2019 +0200

    cook : perform clean_fields() validations

commit e2ffab609d5d618b37c7114397ade48c3b24ad34
Author: Christophe Siraut <csiraut@entrouvert.com>
Date:   Fri Apr 12 14:20:32 2019 +0200

    ServiceBase: validate base_url (#31384)
#8

Mis à jour par Frédéric Péters il y a environ 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF