[python] bearerAuth always type basic

[vmorris]

Description

Defining security component like

"components": {
    "securitySchemes": {
      "bearerAuth": {
        "type": "http",
        "scheme": "bearer"
      }
    }
  },
  "security": [
    {
      "bearerAuth": []
    }
  ]

When I generate a python client, the API configuration code only allows for type: basic.

In configuration.py:

    def auth_settings(self):
        """Gets Auth Settings dict for api client.

        :return: The Auth Settings information dict.
        """
        return {
            'bearerAuth':
                {
                    'type': 'basic',
                    'in': 'header',
                    'key': 'Authorization',
                    'value': self.get_basic_auth_token()
                },

        }

openapi-generator version

3.3.3

[vmorris]

Does #1207 address this?

[vmorris]

Ideally this would be something like

        return {
            'bearerAuth':
                {
                    'type': 'http',
                    'in': 'header',
                    'key': 'Authorization',
                    'value': 'Bearer ' + self.token
                },

        }

[ybelenko]

Does #1207 address this?

As you may already notice. Nope, referenced PR adds token authentication feature to PHP Slim Server generator only.

@tomplus As new Python committee, can you answer this?

[tomplus]

As we can find here /resources/python/configuration.mustache the bearer token will work only if the type equals oauth2.

[vmorris]

I'm not real familiar with mustache.. Shouldn't we be able to add another block for type http? Perhaps a default if oauth2 is not set?

[waza-ari]

In my opinion that should be changed, as the OpenApi allows Bearer authentication without OAuth.

I've tested the proposal by vmorris:

Ideally this would be something like

        return {
            'bearerAuth':
                {
                    'type': 'http',
                    'in': 'header',
                    'key': 'Authorization',
                    'value': 'Bearer ' + self.token
                },

        }

Adding this and of course the token attribute does the job quite well. I'm not too familiar with the template syntax, though

[wing328]

Hi all, I've filed #1999 to add the Bearer authentication support to the Python client. Please review when you've time.

[wing328]

#1999 merged into master. Please try the latest master (snapshot, docker image, etc).

Closing this for the time being.

[waza-ari]

@wing328 sorry it took some time for me to come back to that project. I was testing it today using the current master snapshot docker image and it works perfectly. Thanks for your work!

[esseti]

is this fix released in 3.X version? in my generated code there's no bearerAuth in the auth_setting(self) it should have something like this (note: what should be the identifier ?)

    'bearerAuth':
                {
                    'type': 'http',
                    'in': 'header',
                    'key': 'Authorization',
                    'value': self.get_api_key_with_prefix('<THEPREFIX>')
                },

[vmorris]

@esseti this went into the 4.0.0 milestone

[4c74356b41]

hey, i'm looking at a code generated for python and i see this:

    def auth_settings(self):
        """Gets Auth Settings dict for api client.

        :return: The Auth Settings information dict.
        """
        return {
        }

openapi spec has bearer token auth on all requests pretty much
should I raise a new issue @vmorris ?

[vmorris]

should I raise a new issue @vmorris ?

@4c74356b41 I am not an expert on this group's preferred way to handle this, but I think you should open a new issue and reference this one.