openstack/openstack
Code Issues Proposed changes

Update git submodules

Browse Source 
* Update nova from branch 'master'
  - Merge "Fix target used in nova.policy.check_is_admin"
  - Fix target used in nova.policy.check_is_admin
    
    The target passed to Enforcer.authorize should be a dict,
    similar to the target dict to the RequestContext.can method.
    However, we were passing an instance of _DeprecatedPolicyValues
    because that is ultimately what comes out of
    RequestContext.to_policy_values(). As of change
    I4642c57990b145c0e691140970574412682e66a5 in oslo.policy, that
    incorrect type for the target parameter results in an error in
    the debug logs for the policy check:
    
      cannot format data, exception: Expected a dictionary, got
      <class 'oslo_context.context._DeprecatedPolicyValues'> instead.
    
    This resolves the issue by using the same default target dict
    that RequestContext.can uses if a target is not supplied.
    
    Note that we get here from NovaKeystoneContext via API middleware
    before any request handler is invoked in the wsgi stack, so there
    is no context from the request as to what to pass for the target
    besides the user_id/project_id.
    
    Change-Id: I4442a7b95d15233f76f7795d45b18ac440ddb831
    Closes-Bug: #1807747
This commit is contained in:
Zuul 2019-05-18 02:04:29 +00:00 committed by Gerrit Code Review
parent aac651afbd
commit de855fbb99
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
nova

@ -1 +1 @@
Subproject commit a3cb927fa96f1c3305cacb9a0cf80c2950d1be04
Subproject commit 1d1b0d573671add8630af41754e5521cb2bc5ae1