Merge pull request #239 from emqtt/dev

emqttd_broker:unhook/2 cannot work #238
This commit is contained in:
Feng Lee 2015-08-13 15:39:08 +08:00
commit 6df8acde26
2 changed files with 6 additions and 4 deletions

View File

@ -246,8 +246,8 @@ handle_call({hook, Hook, Name, MFArgs}, _From, State) ->
end,
{reply, Reply, State};
handle_call({unhook, Name}, _From, State) ->
Key = {hook, Name}, Reply =
handle_call({unhook, Hook, Name}, _From, State) ->
Key = {hook, Hook}, Reply =
case ets:lookup(?BROKER_TAB, Key) of
[{Key, Hooks}] ->
ets:insert(?BROKER_TAB, {Key, lists:keydelete(Name, 1, Hooks)});
@ -256,8 +256,9 @@ handle_call({unhook, Name}, _From, State) ->
end,
{reply, Reply, State};
handle_call(_Request, _From, State) ->
{reply, {error, unsupport_request}, State}.
handle_call(Req, _From, State) ->
lager:critical("Unexpected request: ~p", [Req]),
{reply, {error, badreq}, State}.
handle_cast(_Msg, State) ->
{noreply, State}.

View File

@ -308,6 +308,7 @@ shutdown(Error, #proto_state{client_id = undefined}) ->
lager:info("Protocol shutdown ~p", [Error]),
ignore;
%% TODO: ClientId??
shutdown(Error, #proto_state{peername = Peername, client_id = ClientId, will_msg = WillMsg}) ->
lager:info([{client, ClientId}], "Client ~s@~s: shutdown ~p",
[ClientId, emqttd_net:format(Peername), Error]),