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

dd #2

Open
wants to merge 3,558 commits into
base: master
Choose a base branch
from

Conversation

chandramohan-lakshminarayanan
Copy link

@chandramohan-lakshminarayanan chandramohan-lakshminarayanan commented Mar 8, 2018

This change is Reviewable

souravmitra and others added 30 commits February 12, 2016 12:01
 - allows usage of TCP or UDP graphite reporter from metrics lib
…_scanning_hibernate_bundle

Added support for multiple packages in Scanning Hibernate Bundle
Allow selection of `TCP` or `UDP` for Graphite Reporter
add example for Configured Bundles docs
Deal correctly with nested JAR resources
…tion-groups-v2

Allow Validated to annotate non-entity objects
Cannot stop then start appender after upgrading logback beyond 1.1.5.
* Upgrade to the latest Jackson version that correctly handle subtyping
* Remove JDK7 module, because it has been merged to jackson-databind
* Upgrade deprecated API
* Fix the test for printing a detailed YAML parsing error

See the discussion the previous update attempt in issue dropwizard#1449.
The HTTP link returns 301, which breaks maven-deploy plugin.
Al Scott and others added 30 commits August 4, 2016 19:53
…izable-httpclientbuilder

Allow HttpClientBuilder subclasses to customize the underlying builder
…ine-ifs-in-taskservlet

dropwizard-servlets: Combine unnecessarily nested if blocks in TaskServlet
While the improvements to TLS configuration of HTTP clients in 1.0.0
(maybe prior) are awesome, as part of that process the ability to set a
custom HostnameVerifier easily on the HTTP client has been lost.

You used to be able to do e.g. as:

JerseyClientConfiguration myJerseyClientConfiguration = <some
configuration>;
HostnameVerifier verifier = new MyCustomHostnameVerifier();
JerseyClientBuilder clientBuilder = new JerseyClientBuilder(env);
clientBuilder.using(myJerseyClientConfiguration).using(verifier);
Client httpClient = clientBuilder.build();
Same is true for HttpClientBuilder too.

You can still do it by creating a custom Apache
Registry<ConnectionSocketFactory> but you need to set up socket
factories for every scheme.

This change restores the ability to set a custom HostnameVerifier
for clients.

[Fixes dropwizard#1663]
(cherry picked from commit 160502f)
…ropwizard#1677)

ObjectMapper-customization logic is moved from YamlConfigurationFactory
to DefaultConfigurationFactoryFactory, making it easier to modify with a
custom ConfigurationFactoryFactory implementation.

There are some (uncommon) use cases where it's beneficial to update an
application without modifying the configuration, which may fail if the
application configuration has removed a configuration property.

A protected configureObjectMapper method is added to
DefaultConfigurationFactoryFactory to allow the configuration
factory to be extended to apply (or remove) deserialization
features.
…or-provider-shutdown

Gracefully shut down default ExecutorService used by JerseyClientBuilder
…ng/logback-attr-map-race-condition

Fix request attribute-related race condition in Logback request logging
Add ResourceCollection support ServletEnvironment
This in particular makes it possible to configure loggers (whose names
frequently contain periods) with config overrides.

Close dropwizard#1599
A good practice is to avoid manipulating arrays in favour of Java collections,
because arrays are relatively dangerous to operate and provide a less
convenient API.

See Effective Java Item 25: Prefer lists to arrays.
…rray

Use a List instead an array in YamlConfigurationFactory
Instead of listing supported cipher suites, protocols, etc of the default
SSLContext algorithm, log what Jetty's will actually use, which as of this
commit is a "TLS" SSLContext. This will also future proof the logging if
this property ever becomes configurable in Dropwizard.
Document that only TLSv1.2 is enabled by default
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.