search

fcrepo-exts / fcrepo-camel-toolbox

A collection of ready-to-use messaging applications with fcrepo-camel
Apache License 2.0
13 stars 26 forks source link

Update FcrepoLdPathConfig properties #168

Closed mikejritter closed 2 years ago

mikejritter commented 2 years ago

JIRA Ticket: https://fedora-repository.atlassian.net/browse/FCREPO-3774

What does this Pull Request do?

Updates FcrepoLdPathConfig to extend from BasePropsConfig to have consistency for base fcrepo properties

What's new?

Interested parties

@fcrepo/committers

mikejritter commented 2 years ago

Might be good to get some more info in the README about the authHost and authPort to explain what they're for before merging.

dbernstein commented 2 years ago

@mikejritter : this looks good but I'm going to hold off on merging until we get https://github.com/fcrepo-exts/fcrepo-camel-toolbox/pull/165 and https://github.com/fcrepo-exts/fcrepo-camel-toolbox/pull/167 merged.