HttpClient parameterization

Rod Widdowson rdw at steadingsoftware.com
Fri Feb 17 04:14:19 EST 2017


Thanks Brent,

> the most logical place to put that extension class is as an inner class of the parser...at least to me.

I'd gone down that route as well, but as you point out this doesn't buy us anything.  The aim here is to have a single "go to place"
for parameterization for the IdP and that it not be the parser.  I don't see how to square this with defaults being in the base
class over in java-support (except that F3 will help you get there in eclipse).

> Unless you think the dynamic metadata defaults would be elsewhere, but I'm not seeing another logical place.
That’s the rub.  idp-conf\src\main\java springs to mind.

> It probably works, but does start to get a little messy.

Hideous.  If you walk up to it the way we have it comprehensible, but consider someone coming cold to a directory full of class
implementations in some sort of non-obvious hierarchy.  It would need careful javadoc.  Aim number two is ongoing maintainability
after all.

Not withstanding I still like this idea, but I'm not wedded to it and I realize that one's own ideas seem "better".

R



More information about the dev mailing list