Tornado: RedirectHandler.get not taking kwargs

[AbsoluteVirtue]

Hi! Excuse me, if this has been discussed. Search isn't finding anything similar.

Any particular reason, why it's not allowed to do this kind of stuff when using RedirectHandler out of the box?

url(
        pattern=r'/?(?P<lang>ru|en)?/public/(?P<id>\d+)/',
        handler=handlers.Item,
        name='item',
    ),
    url(
        r'/?(?P<lang>ru|en)?/public/(?P<id>\d+)', 
RedirectHandler,
        dict(url='/{lang}/public/{id}/'),
    ),

[bdarnell]

No particular reason, it just hasn't come up before. Patches welcome.

[adamjsawicki]

What is the desired functionality? I'm looking at the implementation, and am unsure where those kwargs would go within the function call.

[bdarnell]

RedirectHandler.get should accept both *args and **kwargs, and pass both of them through to the self._url.format call.

[adamjsawicki]

I can take this.

[bdarnell]

Fixed by #2644