in trunk!

On Nov 13, 8:24 am, "Ray (a.k.a. Iceberg)" <[email protected]> wrote:
> Hi Massimo,
>
> Following code snippet comes from gluon/tools.py 's line 3029~3030.
>
>         elif not request.extension in ('html','load'):
>             raise HTTP(401)
>
> I suggest to change it into:
>
>         elif not request.extension in ('html','load'):
>             raise HTTP(401, str(form.errors()))
>
> This way, client side has a chance to know what goes wrong when error
> happens. That is important when client is visiting the page.json or
> page.xml etc..
>
> At least the suggested modification is harmless. So why not?
>
> Please let me know if you will accept this proposal. Thanks!
>
> Regards,
> Ray

Reply via email to