Edgewall Software
Modify

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#12654 closed defect (fixed)

AuthzPolicy should not raise ConfigurationError when not in permission_policies

Reported by: Ryan J Ollos Owned by: Ryan J Ollos
Priority: normal Milestone: 1.2.1
Component: general Version:
Severity: normal Keywords: authzpolicy, exception
Cc: Branch:
Release Notes:

AuthzPolicy will only raise a ConfigurationError if it's in the list of permission_policies.

API Changes:

Description (last modified by Ryan J Ollos)

After looking at a similar issue for AuthzSourcePolicy in #12637, I realized that AuthzPolicy is incorrectly raising a ConfigurationError under certain conditions when it is enabled but not included in the list of permission_policies. The behavior was implemented in r13741 (#11529), and also #11272 is related.

Attachments (0)

Change History (7)

comment:1 by Ryan J Ollos, 3 years ago

Owner: set to Ryan J Ollos
Status: newassigned

comment:2 by Ryan J Ollos, 3 years ago

Proposed changes in log:rjollos.git:t12654_authzpolicy_configurationerror.

The issue doesn't exist on 1.0-stable.

Last edited 3 years ago by Ryan J Ollos (previous) (diff)

comment:3 by Ryan J Ollos, 3 years ago

Milestone: 1.0.141.2.1

comment:4 by Ryan J Ollos, 3 years ago

Description: modified (diff)

comment:5 by Ryan J Ollos, 3 years ago

Release Notes: modified (diff)
Resolution: fixed
Status: assignedclosed

Committed to 1.2-stable in [15368:15369], merged to trunk in [15370:15371].

comment:6 by Ryan J Ollos, 3 years ago

I'll fix test failures shortly.

comment:7 by Ryan J Ollos, 3 years ago

Fix for test failure committed to 1.2-stable in r15372, merged to trunk in r15373.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted. Next status will be 'reopened'.
to as closed The owner will be changed from Ryan J Ollos to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.