Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Does not support connecting to clusters with basic auth #34

Closed
keety opened this issue Jan 21, 2020 · 1 comment
Closed

Does not support connecting to clusters with basic auth #34

keety opened this issue Jan 21, 2020 · 1 comment

Comments

@keety
Copy link
Contributor

keety commented Jan 21, 2020

The perf-top client does not support connecting to clusters with basic auth.
the options does not support basic auth

perf-top
usage: global.js [-h] --dashboard DASHBOARD [--endpoint ENDPOINT]
                 [--nodename NODENAME] [--logfile LOGFILE]

nor can one pass an endpoint as follows:

 perf-top --endpoint https://username:password@cluster_endpoint --dashboard ClusterOverview
keety added a commit to keety/perftop that referenced this issue Jan 21, 2020
@keety
Add support to pass basic-auth in endpoint url (opendistro-for-elasti…
01fc202 
…csearch#34)

Fixes issue (opendistro-for-elasticsearch#34)  where user can pass basic auth credentials in the
endpoint

example :
```
perf-top --endpoint https://username:password@cluster_endpoint --dashboard ClusterOverview

```
@keety keety mentioned this issue Jan 21, 2020
Merged
@sendkb
Copy link

sendkb commented May 19, 2020

Fixed as a part of #35

@sendkb sendkb closed this as completed May 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@keety @sendkb