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
HTTP Pid Minter #353
HTTP Pid Minter #353
Conversation
* Setup authentication in httpclient. | ||
**/ | ||
private HttpClient buildClient() { | ||
HttpClientBuilder builder = HttpClientBuilder.create().useSystemProperties(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this pool HTTP clients effectively (for high-rate ingest)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure what it does by default. I can add a call to use PoolingHttpClientConnectionManager to be sure it'll pool connections.
… wildcard include, breaking out minter config and making it configurable with system property fcrepo.minter.config
xsi:schemaLocation="http://www.springframework.org/schema/beans http://www.springframework.org/schema/beans/spring-beans-3.0.xsd"> | ||
|
||
<!-- Mints PIDs--> | ||
<bean class="org.fcrepo.kernel.identifiers.UUIDPathMinter" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For ease of use, it may be good to have the configuration for the HttpPidMinter also in this file, but commented out.
…tifier from XML responses using XPath
… wildcard include, breaking out minter config and making it configurable with system property fcrepo.minter.config
Resolved with: 4959897 Addresses: https://www.pivotaltracker.com/story/show/70678050 |
Adding PID minter implementation that retrieves a PID from a remote REST API service.
Fixes https://www.pivotaltracker.com/story/show/70678050