Project

General

Profile

Bug #10813

get_variadic_url supprime le / final dans une URL

Added by Thomas Noël almost 5 years ago. Updated over 4 years ago.

Status:
Fermé
Priority:
Haut
Assignee:
Target version:
Start date:
03 May 2016
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:

Description

L'ajout de ce test plante :

     assert get_variadic_url('[url]foobar/',
            {'url': 'http://www.example.net/'}) == 'http://www.example.net/foobar/'

Le plantage :

E       assert 'http://www.e...le.net/foobar' == 'http://www.ex...e.net/foobar/'
E         - http://www.example.net/foobar
E         + http://www.example.net/foobar/
E         ?                              +


Files

Associated revisions

Revision 89924c19 (diff)
Added by Thomas Noël almost 5 years ago

misc: support variadic URL with trailing / (#10813)

History

#1

Updated by Thomas Noël almost 5 years ago

Alors que celui-ci passe :

    assert get_variadic_url('[url]foo/bar/',
            {'url': 'http://www.example.net/'}) == 'http://www.example.net/foo/bar/'

(juste ajout d'un / entre foo et bar)

#2

Updated by Thomas Noël almost 5 years ago

Pas super évident, voilà où j'en suis dans ma compréhension.

À noter un path.count('/')>=2 qui semblait complètement étrange, remplacé par count('//')==1

(note: ça me gène sur de nouveaux appels webservices famille que je voudrais faire à Alfortville)

#4

Updated by Thomas Noël almost 5 years ago

  • Status changed from En cours to Résolu (à déployer)

Yep, mieux, merci. Je pousse ainsi.

commit 89924c19402073f7eb1aac3c0e7ddc48391ff27f
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Tue May 3 16:59:27 2016 +0200

    misc: support variadic URL with trailing / (#10813)

#5

Updated by Thomas Noël almost 5 years ago

  • Target version set to v1.42
#6

Updated by Frédéric Péters over 4 years ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF