Skip to content
This repository has been archived by the owner on Oct 12, 2019. It is now read-only.

NotificationPluginApi should not throw PaymentPluginApiException #1

Open
pierre opened this issue Oct 2, 2013 · 0 comments
Open

NotificationPluginApi should not throw PaymentPluginApiException #1

pierre opened this issue Oct 2, 2013 · 0 comments
Labels

Comments

@pierre
Copy link
Member

pierre commented Oct 2, 2013

Any runtime exception in a notification plugin will result in the following stacktrace:

Caused by: java.lang.IllegalStateException: Unexpected PaymentApiException for notification plugin
        at com.ning.billing.osgi.bundles.jruby.JRubyNotificationPlugin.handleKillbillEvent(JRubyNotificationPlugin.java:46)
        at com.ning.killbill.osgi.libs.killbill.OSGIKillbillEventDispatcher$1$1.update(OSGIKillbillEventDispatcher.java:64)
        at java.util.Observable.notifyObservers(Observable.java:159)
        at com.ning.billing.osgi.KillbillEventObservable.handleKillbillEvent(KillbillEventObservable.java:61)
        ... 13 more

It looks like we need to introduce a NotificationPluginApiException. It might be tricky to do the dispatch to the right exception in the generator, maybe we also need a generic PluginApiException that we can convert later on from the Java code.

See:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant