yahoo-eng-team team mailing list archive

[Henry Nash <henryn@xxxxxxxxxxxxxxxxxx>]

Due to the fact that we use the msg both for a log and an exception, I
think this code is OK

** Changed in: keystone
       Status: In Progress => Invalid

-- 
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/1443912

Title:
  Non-translation-friendly formatting of msg string

Status in OpenStack Identity (keystone):
  Invalid

Bug description:
   The token controller doesn't adhere to the guidance for string
  translations, instead it pre-formats the substitutions into the msg
  before raising an exception.

  
                  msg = _('User %(u_id)s is unauthorized for tenant %(t_id)s')
                  msg = msg % {'u_id': user_id, 't_id': tenant_id}
                  LOG.warning(msg)
                  raise exception.Unauthorized(msg)

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