Skip to content

Conversation

@jensvog
Copy link
Contributor

@jensvog jensvog commented Jan 5, 2021

Documentation gave information about authentication with key and certificate. I could not find this authentication method in the code and as well not in the Atlassian documentation.

Therefore, I removed the chapter is removed and the other authentication methods are grouped with a new title. Maybe you have more information what could be meant there?

Documentation gave information about authentication with key and certificate.
This method cannot be found in the code and not in the Atlassian
documentation.

Therefore, the chapter is removed and the other authentication methods are
grouped with a new title.
Copy link
Contributor

@Spacetown Spacetown left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGFM because the options are missing in

def __init__(
self,
url,
username=None,
password=None,
timeout=75,
api_root="rest/api",
api_version="latest",
verify_ssl=True,
session=None,
oauth=None,
oauth2=None,
cookies=None,
advanced_mode=None,
kerberos=None,
cloud=False,
proxies=None,
):

@gonchik gonchik merged commit 1b4ca92 into atlassian-api:master Jan 6, 2021
@gonchik
Copy link
Member

gonchik commented Jan 6, 2021

Thank you! :)

@jensvog jensvog deleted the remove_keycert branch January 6, 2021 16:05
at3560k added a commit to at3560k/atlassian-python-api that referenced this pull request Apr 26, 2021
This is nearly a revert of 1b4ca92 / (atlassian-api#682)

The previous information about authentication via key and certificate
remains not directly supported, but the documentation can still
demontsrate how to authenticate.
at3560k added a commit to at3560k/atlassian-python-api that referenced this pull request Apr 26, 2021
This is nearly a revert of 1b4ca92 / (atlassian-api#682)

The previous information about authentication via key and certificate
remains not directly supported, but the documentation can still
demontsrate how to authenticate.
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