manager.notify() doesn't handle the NotImplementedError exceptions

Bug #1690117 reported by Lisa Zangrando on 2017-05-11
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Synergy Service
Low
Lisa Zangrando

Bug Description

In case a manager doesn't implement the method doOnEvent() the NotImplementedError exception is raised. The manager.notify() doesn't handle such exceptions and fails.

Lisa Zangrando (lisa-zangrando) wrote :

Fix #463962

Changed in synergy-service:
assignee: nobody → Lisa Zangrando (lisa-zangrando)
importance: Undecided → Low
status: New → Fix Committed

Reviewed: https://review.openstack.org/463962
Committed: https://git.openstack.org/cgit/openstack/synergy-service/commit/?id=a2ba511da391b87512f1d177304c445fccf8b5a0
Submitter: Jenkins
Branch: master

commit a2ba511da391b87512f1d177304c445fccf8b5a0
Author: Lisa Zangrando <email address hidden>
Date: Thu May 11 12:22:53 2017 +0200

    manager.notify() doesn't handle the NotImplementedError exceptions

    In case a manager doesn't implement the method doOnEvent() the
    NotImplementedError exception is raised. The manager.notify()
    doesn't handle such exceptions and fails. This fix solves the problem.

    Bug: #1690117
    Change-Id: I005658c4341106b54f6757ee4dfcca08a2405605
    Sem-Ver: bugfix

Changed in synergy-service:
status: Fix Committed → Fix Released
Changed in synergy-service:
milestone: none → 1.5.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers