Public bug reported:

Invalid parameter name on interface

There are several classes that inherit from the abstract method
AuthMethodHandler.authenticate. In some cases those classes are
not using matching parameter names.

This patch changes all classes such that the signatures match.
Prior to this there were four different signatures:

authenticate(self, context, auth_payload, auth_context)
authenticate(self, request, auth_info, auth_context)
authenticate(self, request, auth_payload, auth_context)
authenticate(self, request, auth_payload, user_context)

The new common signature should be:

authenticate(self, request, auth_payload, auth_context)

** Affects: keystone
     Importance: Medium
     Assignee: Eric Brown (ericwb)
         Status: Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Identity (keystone).
https://bugs.launchpad.net/bugs/1659730

Title:
  Invalid parameter name on interface

Status in OpenStack Identity (keystone):
  Fix Released

Bug description:
  Invalid parameter name on interface

  There are several classes that inherit from the abstract method
  AuthMethodHandler.authenticate. In some cases those classes are
  not using matching parameter names.

  This patch changes all classes such that the signatures match.
  Prior to this there were four different signatures:

  authenticate(self, context, auth_payload, auth_context)
  authenticate(self, request, auth_info, auth_context)
  authenticate(self, request, auth_payload, auth_context)
  authenticate(self, request, auth_payload, user_context)

  The new common signature should be:

  authenticate(self, request, auth_payload, auth_context)

To manage notifications about this bug go to:
https://bugs.launchpad.net/keystone/+bug/1659730/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to