Skip to content

added: typesafe config integration extras module #1500

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

Closed

Conversation

coutoPL
Copy link
Contributor

@coutoPL coutoPL commented Jan 18, 2018

Motivation:

Current implemenation does not have any integration with popular config library.

Modifications:
Added extras module with Typesafe Config integration and provided AHC config wrapper for mentioned lib.

Result:
AsyncHttpClient can be initialized with Typesafe Config object.

@coutoPL coutoPL closed this Jan 18, 2018
@coutoPL coutoPL reopened this Jan 18, 2018
@@ -0,0 +1,397 @@
/*
* Copyright (c) 2015 AsyncHttpClient Project. All rights reserved.
Copy link
Contributor

Choose a reason for hiding this comment

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

2018

@@ -0,0 +1,121 @@
/*
* Copyright (c) 2015 AsyncHttpClient Project. All rights reserved.
Copy link
Contributor

Choose a reason for hiding this comment

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

2018

@slandelle
Copy link
Contributor

@coutoPL Could you please:

Thanks!

Current implemenation does not have any integration with popular config library.

Modifications:
Added extras module with Typesafe Config integration and provided AHC config wrapper for mentioned lib.

Result:
AsyncHttpClient can be initialized with Typesafe Config file.
@coutoPL coutoPL force-pushed the feature/typesafeconfig branch from 029fa53 to a220b71 Compare January 18, 2018 21:33
slandelle added a commit that referenced this pull request Jan 19, 2018
Motivation:

Bump minor version before merging #1500 that brings a new feature

Modification:

Bump to 1.2.0-SNAPSHOT

Result:

New minor version
@slandelle
Copy link
Contributor

cherry-picked as 60a8fe2

Thanks!

@slandelle slandelle closed this Jan 19, 2018
@slandelle slandelle added this to the 2.2.0 milestone Jan 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants