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

Possibility to provide custom data loaders #24

Closed
wants to merge 1 commit into from

Conversation

rvillido
Copy link
Contributor

This change adds the possibility to provide your own data loaders so it would be possible to fine tune network connections to OCSP, TSA and TSL services.

Some integrators would like to control how the network connections are made to fine tune the performance, proxy configurations, timeouts, headers etc (instead of every integrator asking DigiDoc4j team to implement it for them).

It is now possible to create a new DataLoaderFactory for each remote service (OCSP, TSA, TLS) that creates DataLoader instances that control how network connections are made. The factories can be set in the Configuration class.

Here is an example of creating your own data loader for OCSP, TimeStamp and TSL requests:

// Implement a new data loader factory that creates custom data loaders
public class MockDataLoaderFactory implements DataLoaderFactory {
@OverRide
public DataLoader create() {
return new MockDataLoader();
}
}
MockDataLoaderFactory mockDataLoaderFactory = new MockDataLoaderFactory();

// Using it for OCSP requests
configuration.setOcspDataLoaderFactory(mockDataLoaderFactory);
// Using it for TimeStamp requests
configuration.setTimestampDataLoaderFactory(mockDataLoaderFactory);
// Using it for TSL requests
configuration.setTslDataLoaderFactory(mockDataLoaderFactory);

…ne network connections to OCSP, TSA and TSL services
@naare
Copy link
Contributor

naare commented Dec 18, 2019

This feature is implemented with DD4J-520 as it was not possible to merge the pull request directly.

@naare naare closed this Dec 18, 2019
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.

None yet

2 participants