Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Set user credentials when present in the query endpoint URI and set SSL based on URI scheme #65

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Set user credentials when present in the query endpoint URI and set SSL based on URI scheme #65

wants to merge 8 commits into from

Conversation

lkitching
Copy link
Contributor

No description provided.

@lkitching
Set basic authentication credentials for users in query endpoint URIs.
573621a 
Issue #63 - if user information is present on the query endpoint URI, set
the corresponding basic authentication credentials on outgoing query
requests.
@lkitching
Use query endpoint URI scheme to configure SSL.
5b21ac1 
Issue #64 - Use SSL for SPARQL queries where the scheme of the query
endpoint URI is https rather than expecting the default SSL port of
443.
@lkitching lkitching requested a review from ricroberts August 2, 2017 15:00
@ricroberts
Copy link
Contributor

@lkitching can you have a look at the travis build failures - its having trouble connecting with these changes.

@lkitching
Copy link
Contributor Author

@RicSwirrl - Travis build is now passing

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

@ricroberts ricroberts Awaiting requested review from ricroberts

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lkitching @ricroberts