Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kazoo 80 #125

Merged
merged 6 commits into from Oct 18, 2013
Merged

Kazoo 80 #125

merged 6 commits into from Oct 18, 2013

Conversation

k-anderson
Copy link
Contributor

No description provided.

@k-anderson
Copy link
Contributor Author

Before accepting please test originate ;)

jamesaimonetti pushed a commit that referenced this pull request Oct 18, 2013
Kazoo 80 - Gangsta! Looking forward to rolling this puppy out!
@jamesaimonetti jamesaimonetti merged commit e35683f into master Oct 18, 2013
@lazedo
Copy link
Member

lazedo commented Oct 18, 2013

Hi,
this isn't working for originate requests like the one made by fax_worker {<<"Resource-Type">>, <<"originate">>} , it does work with {<<"Resource-Type">>, <<"audio">>}
i tried to debug this and changed
<<"Call-ID">> to <<"Outbound-Call-ID">> in stepswitch_request_sup:originate and stepswitch_originate:start_link

it seems that the originate_req is getting a lot of info from stepswitch:resources and fails to build the api message.
the REQ_HEADER <> has multilevel proplist (changed from to_proplist to recursive_to_proplist) but it fails when it tries to rebuild the wh_json:from_proplist since this doesn't handle multilevel.

this is where it fails stepswitch_originate
handle_cast({'gen_listener', {'is_consuming', 'true'}}, State) ->
'ok' = wapi_resource:publish_originate_req(build_originate(State)),

it fails in wh_api
headers_to_json([|]=HeadersProp) ->
try wh_json:encode(wh_json:from_list(HeadersProp)) of ====>>>> fails here
JSON -> {'ok', JSON}
catch
_What:_Why ->
lager:info("WHISTLE TO_JSON p: pnpn", [_What, _Why, HeadersProp]),
{'error', io_lib:format("WHISTLE TO_JSON p: pnp
n", [_What, _Why, HeadersProp])}
end.

please check this.
cheers

@k-anderson k-anderson deleted the KAZOO-80 branch June 5, 2014 23:29
sudomabider pushed a commit to OpenTelecom/kazoo that referenced this pull request Oct 16, 2018
[KAZ-576] Merge Recent 3.22 Changes into 4.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants