Synergy Service

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

Bug #1690117 reported by Lisa Zangrando
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Synergy Service
Fix Released
Low
Lisa Zangrando
Synergy Service 1.5.2

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.

Revision history for this message
Lisa Zangrando (lisa-zangrando) wrote :

Fix #463962

Changed in synergy-service:
assignee: nobody → Lisa Zangrando (lisa-zangrando)
importance: Undecided → Low
status: New → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to synergy-service (master)

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
Lisa Zangrando (lisa-zangrando)
Changed in synergy-service:
milestone: none → 1.5.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.