Skip to content
This repository has been archived by the owner on Jan 3, 2019. It is now read-only.

Added authenticated access to Fedora repo. #36

Closed
wants to merge 11 commits into from

Conversation

nikhiltri
Copy link
Contributor

I accidentally committed password credentials to my previous fork, so I had to delete the repo from Github. This pull request represents the same work, but unfortunately without a log of my previous commits. Sorry about that!

@nikhiltri
Copy link
Contributor Author

FYI, this addresses the following issue: https://www.pivotaltracker.com/s/projects/684825/stories/64929156


## Authenticated repo

If REST calls to you Fedora repository require BASIC authentication,
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: "you" -> "your"

@@ -210,10 +240,21 @@ public void onMessage(final Message message) {
* Index a resource.
**/
private void index( final String uri, final String eventType ) {
// If the Fedora instance requires authentication, set it up here
if (this.fedoraUsername != null && !"".equals(this.fedoraUsername)) {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From:
if (this.fedoraUsername != null && !"".equals(this.fedoraUsername)) {

To:
if (!StringUtils.isBlank(fedoraUsername) && !StringUtils.isBlank(this.fedoraPassword)) {

@awoods
Copy link

awoods commented Apr 30, 2014

Resolved with: b89273c

Addresses: https://www.pivotaltracker.com/story/show/64929156

@awoods awoods closed this Apr 30, 2014
@awoods awoods mentioned this pull request Apr 30, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants