Response to PR comments
backed out whitespace changes
various access modifiers and logging details
removed MockHeaderFilter, will use normal mocks or ITs to test principal factory functions