Skip to content

Conversation

jun06t
Copy link
Contributor

@jun06t jun06t commented Oct 6, 2016

I sometimes get following error.

TypeError Cannot read property 'EndpointArn' of null 
    /usr/local/parse/node_modules/parse-server-sns-adapter/src/SNSPushAdapter.js:176:21 getPlatformArn
    /usr/local/parse/node_modules/parse-server-sns-adapter/node_modules/aws-sdk/lib/request.js:354:18 none
    /usr/local/parse/node_modules/parse-server-sns-adapter/node_modules/aws-sdk/lib/sequential_executor.js:105:20 callListeners
    /usr/local/parse/node_modules/parse-server-sns-adapter/node_modules/aws-sdk/lib/sequential_executor.js:77:10 emit
    /usr/local/parse/node_modules/parse-server-sns-adapter/node_modules/aws-sdk/lib/request.js:596:14 emit
    /usr/local/parse/node_modules/parse-server-sns-adapter/node_modules/aws-sdk/lib/request.js:21:10 transition
    /usr/local/parse/node_modules/parse-server-sns-adapter/node_modules/aws-sdk/lib/state_machine.js:14:12 runTo

to prevent this i fixed the code.

@rogerhu
Copy link
Collaborator

rogerhu commented Oct 6, 2016

Do you happen to have a test you could add?

@jun06t
Copy link
Contributor Author

jun06t commented Oct 6, 2016

i'm sorry. i've added a test.

@rogerhu
Copy link
Collaborator

rogerhu commented Oct 6, 2016

Lgtm thanks

@rogerhu rogerhu merged commit 2547ddf into parse-server-modules:master Oct 6, 2016
@jun06t jun06t deleted the arn branch October 7, 2016 01:17
@madsb madsb mentioned this pull request Jan 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants