Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix authType vs AuthType var discrepancy #343

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

xbezdick
Copy link

No description provided.

@ankitpokhrel
Copy link
Owner

Hi @xbezdick, Thank you for the PR!

I see that you changed unexported variables to the exported one. Could you please explain the reason for that? I don't see any discrepancies here.

@xbezdick
Copy link
Author

Hi, the unexported variable in go in for example Fedora is not available to client.go. It works on mac for some reason but will not work on most linux machines I tested on.

@dofinn
Copy link

dofinn commented Jun 18, 2022

Confirmed this - main doesnt work on F36, but this PR does. Otherwise it defaults to "Basic" auth, regardless of setting env.

@ankitpokhrel
Copy link
Owner

This is strange!

@xbezdick @danobi Can we confirm that JIRA_AUTH_TYPE variable is accessible in the shell/sub-shell and has a correct value? It looks like the variable is somehow not available in the shell. Do you have same issue if you set auth type directly in the config: auth_type: bearer?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants