Skip to content
This repository was archived by the owner on Oct 20, 2022. It is now read-only.

added token auth method#5

Open
ttaeschn wants to merge 1 commit into
Orange-OpenSource:masterfrom
ttaeschn:heads/addTokenAuthMethod
Open

added token auth method#5
ttaeschn wants to merge 1 commit into
Orange-OpenSource:masterfrom
ttaeschn:heads/addTokenAuthMethod

Conversation

@ttaeschn

Copy link
Copy Markdown

implements #3

@Karreg

Karreg commented Feb 24, 2020

Copy link
Copy Markdown
Member

Hey, thanks @ttaeschn for the PR!
Especially since I don't have time to work on this plugin at the moment...

Still, I would rather use another proeprty, having a token in the "authPath" variable is counter-intuitive. Maybe something like this:

if authToken is defined
  then use authToken
  else use authPath

WDYT?

@osminm

osminm commented Feb 17, 2021

Copy link
Copy Markdown

Was this authToken feature implemented? Trying to use this in an environment with multiple namespaces and don't see how I can specify a namespace but a token might work.

@Karreg

Karreg commented Feb 25, 2021

Copy link
Copy Markdown
Member

not yet, I'll try to work on it today. your work will help.

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.

3 participants