Skip to content
This repository was archived by the owner on Jan 18, 2024. It is now read-only.

Use auth for search requests #53

Merged
merged 1 commit into from
May 23, 2014
Merged

Use auth for search requests #53

merged 1 commit into from
May 23, 2014

Conversation

@laurentstemmermd
Copy link
Contributor

@laurentstemmermd laurentstemmermd commented May 23, 2014

It seems like you are missing auth: opts.sourceAuth or auth: opts.targetAuth each time http.request is performed. So it's not possible to use the exporter with basic auth without fixing it manualy.

http.request({
            host : opts.sourceHost,
            port : opts.sourcePort,
            auth: opts.sourceAuth,

@laurentstemmermd laurentstemmermd mentioned this pull request May 23, 2014
Closed
mallocator added a commit that referenced this pull request May 23, 2014
@mallocator
Merge pull request #53 from laurentstemmermd/hotfix/use_auth_for_requ…
30a7d90 
…ests

Use auth for search requests
@mallocator mallocator merged commit 30a7d90 into mallocator:master May 23, 2014
@mallocator
Copy link
Owner

mallocator commented May 23, 2014

Looks good, thanks :)

@mallocator mallocator added the bug label May 23, 2014
@mallocator mallocator added this to the 1.3.2 milestone May 23, 2014
@mallocator mallocator self-assigned this May 23, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

@mallocator mallocator

Labels

bug

Projects

None yet

Milestone

1.3.2

Development

Successfully merging this pull request may close these issues.

2 participants

@laurentstemmermd @mallocator