Assign check_headers decorator as class declaration to prevent Recursion Error

[philof]

Using a class decoration in Flask assigns the decorator at instantiation rather than each request, which @kypsibir pointed out.

Every request is now stable.

--------------------------------------------------------------------------------
127.0.0.1 - - [08/Apr/2017 15:55:06] "GET /persons HTTP/1.1" 200 -
--------------------------------------------------------------------------------
PATH: '/persons'
         4749 function calls (4673 primitive calls) in 0.017 seconds

[coveralls]

Coverage decreased (-0.008%) to 95.965% when pulling 81a2c85226fac08f5ffebf1a8a906fcf7c8fbe21 on philof:master into 8597f8c89a73bcc0d331155a5c4b38801026d2b6 on miLibris:master.

[coveralls]

Coverage decreased (-0.008%) to 95.965% when pulling 81a2c85226fac08f5ffebf1a8a906fcf7c8fbe21 on philof:master into 8597f8c89a73bcc0d331155a5c4b38801026d2b6 on miLibris:master.

[philof]

27

[akira-dev]

I can't do this because if you set decorators in the end class this decorator list in the base resource will be overridden so check_headers will never be performed. I will make a fix for this today with metaclass to construct the class dynamically instead of using the constructor the instance.

[philof]

That use case is already tested in the unit tests. Test_wrong_accept_headers hits PersonList which has custom get and post decorators.

[akira-dev]

New release is available on pypi with bug fix of this issue

[akira-dev]

Thank you very much for your contribution