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.

Sign up for GitHub

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

Jump to bottom

MandrillRequestDispatcher.execute throwing NPE #88

Open
katangagonzalez opened this issue Jun 5, 2018 · 5 comments
Open

MandrillRequestDispatcher.execute throwing NPE #88

katangagonzalez opened this issue Jun 5, 2018 · 5 comments

Comments

@katangagonzalez
Copy link

After upgrading to version 0.0.8 I started to have some NPE at line 124 of MandrillRequestDispatcher, this is because response is null at this point. Is there any workaround for this or a way to debug my request in order to figure out what is going on? Many thanks.
@katangagonzalez
Copy link
Author

Please, check this commit where null check was removed

@billoneil
Copy link
Collaborator

Hmm that commit looks like it could be the culprit but I'm not sure how the response is actually null there. Are you using a proxy or anything?

@katangagonzalez
Copy link
Author

No, I assume that call to execute threw an exception. Would you recommend to come back to 0.0.7 version?

@ndemengel
Copy link

Hi, I'm experiencing the same problem for some requests. Could you please restore the null check?

pv-pavan pushed a commit to pv-pavan/lutung that referenced this issue Jul 18, 2020
@pavan-mt
MandrillRequestDispatcher.execute throwing NPE rschreijer#88
46093e7
@pv-pavan
Copy link

#103 This shud fix.

dnovitski referenced this issue in dnovitski/lutung Sep 29, 2020
@dnovitski
Merge pull request #1 from pv-pavan/master
36bd78d 
MandrillRequestDispatcher.execute throwing NPE #88
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ndemengel @pv-pavan @billoneil @katangagonzalez