Flacon de rediriger plusieurs itinéraires

Je suis en train de mettre en œuvre une redirection modèle, similaire à ce que StackOverflow n':

@route('/<int:id>/<username>/')
@route('/<int:id>/')
def profile(id, username=None):
    user = User.query.get_or_404(id)

    if user.clean_username != username:
        return redirect(url_for('profile', id=id, username=user.clean_username))

    return render_template('user/profile.html', user=user)

Voici un tableau simple de ce qui devrait se produire:

URL                         Redirects/points to
====================================================
/user/123                   /user/123/clean_username
/user/123/                  /user/123/clean_username
/user/123/foo               /user/123/clean_username
/user/123/clean_username    /user/123/clean_username
/user/123/clean_username/   /user/123/clean_username/
/user/125698                404

Maintenant, je peux accéder au profil avec /user/1/foo, mais /user/1 produit un BuildError. J'ai essayé de la alias=True argument mot-clé et quelque chose avec defaults, mais je ne suis pas tout à fait sûr ce n'est pas de travail.

Comment aurais-je un itinéraire rediriger vers l'autre?

Je ne sais pas si je suis absent quelque chose.. mais qui ne devrait pas l'être à l'intérieur d'une classe et d'avoir un self paramètre? (en supposant que vous utilisez flacon chic) Autre chose qui pourrait être mal si vous êtes en utilisant des modèles est que vous devez ajouter le modèle de nom de url_for (flask.pocoo.org/docs/blueprints/#building-urls).
Si vous êtes l'obtention d'un BuildError ensuite, il ya quelque chose de mal avec votre appel à url_for. Pouvez-vous fournir le traceback?
C'est un exemple simplifié qui illustre bien le problème. À l'aide de plans ou de Flacon Chic n'aurait pas vraiment changer quoi que ce soit.
vous pouvez poster votre app.url_map? cela devrait vous aider à comprendre ce qui fait url_for de s'étouffer.
Donc, euh... que l'on devrait obtenir le bounty?
Eh bien, il s'est avéré que mon problème était dû à une erreur stupide de ma part, donc je ne suis pas sûr.
Si nous sommes d'être littérale, je ne crois strict_slashes fait la différence ici, /user/123/foo redirige vers /user/123/foo/. c'est le /user/123/foo/ url, ce qui n'apparaît pas dans votre tableau, qui exécutera votre fonction et est censé rediriger vers la bonne url.

InformationsquelleAutor Blender | 2013-06-24

le débogage des itinéraires de:

Mise à jour: répondre à la principale question "quel est le problème avec mes parcours", de la façon la plus simple de debug qui est de l'utilisation app.url_map; e.g:

>>> app.url_map
Map([<Rule '/user/<id>/<username>/' (HEAD, OPTIONS, GET) -> profile>,
 <Rule '/static/<filename>' (HEAD, OPTIONS, GET) -> static>,
 <Rule '/user/<id>/' (HEAD, OPTIONS, GET) -> profile>])

Dans ce cas, cela confirme que le point de terminaison est correctement réglé.
Voici un exemple mettant en valeur à la fois simple flask et flask-classy:

from app import app, models
from flask import g, redirect, url_for, render_template, request
from flask.ext.classy import FlaskView, route

@app.route('/user/<int:id>', strict_slashes=False)
@app.route('/user/<int:id>/<username>', strict_slashes=False)
def profile(id, username=None):
    user = models.User.query.get_or_404(id)
    if user.clean_username != username:
        return redirect(url_for('profile', id=id, username=user.clean_username))
    return render_template('profile.html', user=user)

class ClassyUsersView(FlaskView):
    @route('/<int:id>', strict_slashes=False)
    @route('/<int:id>/<username>', strict_slashes=False, endpoint='classy_profile')
    def profile(self, id, username=None):
        user = models.User.query.get_or_404(id)
        if user.clean_username != username:
            return redirect(url_for('classy_profile', id=id, username=user.clean_username))
        return render_template('profile.html', user=user)

ClassyUsersView.register(app)

Ils ont différents points de terminaison, qui vous devez prendre en compte pour url_for:

>>> app.url_map
Map([<Rule '/classyusers/<id>/<username>' (HEAD, OPTIONS, GET) -> classy_profile>,
 <Rule '/user/<id>/<username>' (HEAD, OPTIONS, GET) -> profile>,
 <Rule '/classyusers/<id>' (HEAD, OPTIONS, GET) -> ClassyUsersView:profile_1>,
 <Rule '/static/<filename>' (HEAD, OPTIONS, GET) -> static>,
 <Rule '/user/<id>' (HEAD, OPTIONS, GET) -> profile>])

Sans flask-classy le nom du point de terminaison est le nom de la fonction, mais comme vous l'avez constaté, ce qui est différent lors de l'utilisation de classy, et vous pouvez regarder le nom du point de terminaison avec url_map() ou de céder votre itinéraire avec @route(..., endpoint='name').

moins redirections:

Pour répondre à l'url que vous avez posté, tout en minimisant la quantité de redirection, vous devez utiliser strict_slashes=False, ce sera assurez-vous de traiter les demandes qui ne sont pas terminées avec un / au lieu de les rediriger avec un 301 rediriger vers leur /terminée par contrepartie:

@app.route('/user/<int:id>', strict_slashes=False)
@app.route('/user/<int:id>/<username>', strict_slashes=False)
def profile(id, username=None):
    user = models.User.query.get_or_404(id)
    if user.clean_username != username:
        return redirect(url_for('profile', id=id, username=user.clean_username))
    return render_template('profile.html', user=user)

voici le résultat:

>>> client = app.test_client()
>>> def check(url):
...     r = client.get(url)
...     return r.status, r.headers.get('location')
... 
>>> check('/user/123')
('302 FOUND', 'http://localhost/user/123/johndoe')
>>> check('/user/123/')
('302 FOUND', 'http://localhost/user/123/johndoe')
>>> check('/user/123/foo')
('302 FOUND', 'http://localhost/user/123/johndoe')
>>> check('/user/123/johndoe')
('200 OK', None)
>>> check('/user/123/johndoe/')
('200 OK', None)
>>> check('/user/125698')
('404 NOT FOUND', None)

Comportement de strict_slashes:

with strict_slashes=False
URL                         Redirects/points to              # of redirects
===========================================================================
/user/123                   302 /user/123/clean_username          1
/user/123/                  302 /user/123/clean_username          1
/user/123/foo               302 /user/123/clean_username          1
/user/123/foo/              302 /user/123/clean_username          1
/user/123/clean_username    302 /user/123/clean_username          1
/user/123/clean_username/   200 /user/123/clean_username/         0
/user/125698                404
with strict_slashes=True (the default)
any non '/'-terminated urls redirect to their '/'-terminated counterpart
URL                         Redirects/points to              # of redirects
===========================================================================
/user/123                   301 /user/123/                        2
/user/123/foo               301 /user/123/foo/                    2
/user/123/clean_username    301 /user/123/clean_username/         1
/user/123/                  302 /user/123/clean_username/         1
/user/123/foo/              302 /user/123/clean_username/         1
/user/123/clean_username/   200 /user/123/clean_username/         0
/user/125698                404
example:
"/user/123/foo" not terminated with '/' -> redirects to "/user/123/foo/"
"/user/123/foo/" -> redirects to "/user/123/clean_username/"

Je crois que c'est exactement ce que votre test de la matrice est sur 🙂

Merci, c'est très utile. Je ne connaissais pas strict_slashes soit.

InformationsquelleAutor dnozay

21

Vous avez presque obtenu. defaults est ce que vous voulez. Voici comment cela fonctionne:
```
@route('/<int:id>/<username>/')
@route('/<int:id>/', defaults={'username': None})
def profile(id, username):
user = User.query.get_or_404(id)
if username is None or user.clean_username != username:
return redirect(url_for('profile', id=id, username=user.clean_username))
return render_template('user/profile.html', user=user)
```
defaults est un dict avec des valeurs par défaut pour tous les paramètres de la route qui ne sont pas à la règle. Ici, dans la deuxième route décorateur, il n'y a pas de username paramètre de la règle, de sorte que vous avez à mettre dans defaults.
- Pour info pour quelqu'un d'autre, si vos paramètres par défaut sera le même pour toutes les voies pour la même fonction, vous pouvez définir les valeurs par défaut dans les paramètres de la définition de la fonction à la place des valeurs par défaut dictionnaire de chaque itinéraire décorateur. De cette façon, vous spécifiez uniquement dans un seul endroit.
InformationsquelleAutor dAnjou
3

Bien, il ressemble mon code d'origine a effectivement travaillé. Flacon Chic a la question ici (et depuis que cette question a un bounty, je ne peux pas le supprimer).

J'ai oublié que le Flacon Chic renomme les routes, donc au lieu de url_for('ClassName:profile'), je dois sélectionner le ultrapériphériques décorateur route:
```
url_for('ClassName:profile_1')
```
Une autre solution serait de spécifier explicitement un point de terminaison de la route:
```
@route('/<int:id>/<username>/', endpoint='ClassName:profile')
```
- oui, je me doutais bien de quelque chose comme ça, d'où la question sur l'affichage de votre app.url_map.
- Merci, je ne savais jamais sur app.url_map. Il est même assez imprime les routes!
InformationsquelleAutor Blender
1

Je ne comprends pas pourquoi vous rediriger. Vous n'avez pas gagner quoi que ce soit avec la redirection, et comme vous l'avez mentionné vous-même, vous vous retrouvez juste l'interrogation de la base de données plusieurs fois. Vous n'utilisez pas le nom d'utilisateur de manière significative, donc, juste l'ignorer.
```
@route('/<int:id>/<username>/')
@route('/<int:id>/')
def profile(id, username=None):
user = User.query.get_or_404(id)
return render_template('user/profile.html', user=user)
```
Cela permettra de satisfaire tous vos exemples donnés.
- Reddit est-il de cette façon, mais j'aime StackOverflow approche un peu plus. De toute façon, c'est juste mon exemple. Je suis à la recherche de plus d'une solution à la route de problème de redirection que pour mon cas d'utilisation spécifiques.
InformationsquelleAutor Michael Davis

Vous devez vous connecter pour publier un commentaire.