dgraph-io / ratel

Dgraph Data Visualizer and Cluster Manager
Apache License 2.0
120 stars 49 forks source link

Adding support for X-Dgraph-AuthToken header #248

Closed elliotmassen closed 3 years ago

elliotmassen commented 3 years ago

This PR addresses the issue raised here (and originally in #224) by adding a new "Auth Token" setting to the "Extra Settings" tab in the server connection modal. The inputted value will then be sent as the X-Dgraph-AuthToken HTTP header on requests to the server.

I'm not familar with the Ratel codebase, so it's possible that I've missed something! I've followed the same pattern used for the Slash API Key for consistency.

When I ran ./scripts/build.prod.sh it made updates to yarn.lock and package-lock.json, however I presumed these changes should not be commited as this PR does not introduce any new depedencies.


This change is Reviewable

Ratel Preview: Dgraph Preview

CLAassistant commented 3 years ago

CLA assistant check
All committers have signed the CLA.

MichelDiz commented 3 years ago

@elliotmassen can you sign the CLA?

elliotmassen commented 3 years ago

@MichelDiz Done :)

MichelDiz commented 3 years ago

@elliotmassen thanks for the PR. We'll see it live as soon we have a Ratel release.

Cheers.