yahoo-eng-team team mailing list archive

[Brant Knudson <bknudson@xxxxxxxxxx>]

Public bug reported:


The Federation manager's get_mapping_from_idp_and_protocol method is returning a dict where the 'rules' value is a JSON string. That 'rules' is stored as a JSON string is an implementation detail that shouldn't be exposed to callers.

The 'rules' value of the dict returned by
get_mapping_from_idp_and_protocol should be JSON-parsed before it's
returned so that callers don't have to know that it's a JSON string.
This way all callers don't have to parse the string themselves.

** Affects: keystone
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to Keystone.
https://bugs.launchpad.net/bugs/1469517

Title:
  Federation get_mapping_from_idp_and_protocol should return object

Status in OpenStack Identity (Keystone):
  New

Bug description:
  
  The Federation manager's get_mapping_from_idp_and_protocol method is returning a dict where the 'rules' value is a JSON string. That 'rules' is stored as a JSON string is an implementation detail that shouldn't be exposed to callers.

  The 'rules' value of the dict returned by
  get_mapping_from_idp_and_protocol should be JSON-parsed before it's
  returned so that callers don't have to know that it's a JSON string.
  This way all callers don't have to parse the string themselves.

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