1147255 - There's a missing logger for webapi_logger_handler

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Certification Suite, defect)

Description

[Al Tsai [:atsai]]

https://github.com/mozilla-b2g/fxos-certsuite/blob/master/certsuite/cert.py#L80

There's a logger missing.

Comment 1

[oo[oouyang]]

There should be an global 'logger' and it's set in 'cli' function and can be used in the 'webapi_log_handler' function.
The global 'logger' declare is missed.

Comment 2

[oo[oouyang]]

Attachment: add global logger declare

Comment 3

[Askeing Yen[:askeing][:fyen]]

Comment on attachment 8582848 [details] [review]
add global logger declare

LGTM

Comment 4

[Askeing Yen[:askeing][:fyen]]

https://github.com/mozilla-b2g/fxos-certsuite/commit/70c6fd9b40714e40337e66ae2edcd14db09d84e9

Comment 5

[Al Tsai [:atsai]]

Comment on attachment 8582848 [details] [review]
add global logger declare

clean flag. thanks :D