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

Complete operations with a failure if deserializing response fails #27

Merged
merged 5 commits into from
Mar 26, 2020
Merged

Complete operations with a failure if deserializing response fails #27

merged 5 commits into from
Mar 26, 2020

Conversation

dontgitit
Copy link

Fixes #26

Requests would previously never complete if an exception was thrown by the deserializer

"Typesafe repository releases" at "https://repo.typesafe.com/typesafe/releases/"
"Typesafe repository releases" at "https://repo.typesafe.com/typesafe/releases/",
// using staging artifact until 2.13 release is ready: https://github.com/apache/logging-log4j-scala/pull/3
"Apache staging repository" at "https://repository.apache.org/content/repositories/staging"
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure how you want to handle this, but vimalaguti/logging-log4j-scala is gone so it can no longer pull in the log4jScala project above

Copy link
Owner

@Ma27 Ma27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change seems fine, thanks!
Can you please take a look at the test failure (cannot reproduce it when running sbt clean coverage +test locally multiple times).

Otherwise I may take a look at it this weekend :)

@dontgitit
Copy link
Author

Right, I don't think the failure is related to my change? Any way you can retrigger the build on travis?

@Ma27
Copy link
Owner

Ma27 commented Mar 24, 2020

Already tried it this afternoon (in European time), a new build is running now, let's see if it helps this time :)

@dontgitit
Copy link
Author

Looks like the rebuild passed? But the status on the PR still says failed...

@Ma27
Copy link
Owner

Ma27 commented Mar 26, 2020

You're right 🤦‍♂️

@Ma27 Ma27 merged commit e854377 into Ma27:master Mar 26, 2020
@Ma27
Copy link
Owner

Ma27 commented Mar 26, 2020

@dontgitit thanks! Will hopefully be able to publish a release this weekend (in case I forget about it, feel free to ping me :))

@Ma27
Copy link
Owner

Ma27 commented May 9, 2020

Sorry for the delay, just published 1.9.1. It should be available on Maven central pretty soon, already released the artifacts in Sonatype's Nexus :)

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.

Requests never completed if decoding fails
2 participants