chris-ricketts / actix-broker Goto Github PK

like as a PR that fixes actix/actix#256?

Hi,

With actix-web getting stable and using actix 0.8.2, actix-broker can no longer be used with the latest version since it depends on actix 0.7.9.

Bumping the version to actix 0.8.2 and updating the examples should be straightforward. I'll submit a PR.

Motivation:
The vast majority of my actix-web application is implemented as actors, however there are some parts which are more convenient to implement as normal or async methods.

I want to be able to notify running actors of something that happened in a normal method, but there is no actor context.

Workaround:
Create a sentinel actor that sends the message (generic version untested):

struct Notifier<T>(T);

impl<T: Message> Actor for Notifier<T> {
    type Context = Context<Self>;

    fn started(&mut self, ctx: &mut Self::Context) {
        self.issue_system_async(self.0);
        ctx.stop();
    }
}

As I am no longer actively developing actix-broker it was suggested by @robjtede that it be moved to the official actix-extras repo.

@robjtede what can I do to help?