PR#3982: Only print the traceback on debug or non-pre-receive hooks -

Fixes https://pagure.io/pagure/issue/3974

Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr

@bkabrda since you reported the issue, do you have ways to confirm this works as expected?

@pingou yes, I'll try the patch out and let you know. Thanks for working on this!

@pingou I'd suggest printing str(e) in an else clause, because otherwise the users would see no reason why the push was refused.

rebased onto bd83f76eecce2ca1bb50aac4887b496a6d3f6a30

Nov 5, 2018

@bkabrda good catch, adjusted :)

rebased onto 7716e2dc15cf85e3ca457b402839489e272a7960

Nov 5, 2018

:thumbsup:

Cool, thanks! :)

Pull-Request has been merged by pingou

Nov 5, 2018

Assignee

None

Tags

No Tags