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

FHIR Tester application use configured client factory #211

Merged
merged 2 commits into from Aug 25, 2015

Conversation

Projects
None yet
2 participants
@pukkaone
Contributor

pukkaone commented Aug 24, 2015

I want the FHIR Tester application to connect to a FHIR server that requires HTTP Basic authentication. I followed the instructions at http://jamesagnew.github.io/hapi-fhir/doc_server_tester.html but setting the clientFactory property to my custom ITestingUiClientFactory implementation had no effect. The FHIR Tester application displayed the error message:

Failed to load conformance statement, error was: ca.uhn.fhir.rest.server.exceptions.AuthenticationException: HTTP 401 Unauthorized

This pull request changes the FHIR Tester application to actually use the configured client factory as documented.

pukkaone added some commits Aug 24, 2015

Create client from configured client factory
In the FHIR Tester application, actually use the client factory property
configured in the tester configuration.
@jamesagnew

This comment has been minimized.

Owner

jamesagnew commented Aug 25, 2015

Thanks for the contribution!

Merging now.

jamesagnew added a commit that referenced this pull request Aug 25, 2015

Merge pull request #211 from pukkaone/tester-client-factory
FHIR Tester application use configured client factory

@jamesagnew jamesagnew merged commit d6540a9 into jamesagnew:master Aug 25, 2015

1 of 2 checks passed

coverage/coveralls Coverage decreased (-0.01%) to 81.929%
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

jamesagnew added a commit that referenced this pull request Aug 25, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment