[SERVER-29314] fix cyclic dependency between commands and audit libraries Created: 22/May/17 Updated: 30/Oct/23 Resolved: 14/Jun/17 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Security |
| Affects Version/s: | None |
| Fix Version/s: | 3.5.8 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Benety Goh | Assignee: | Benety Goh |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Backwards Compatibility: | Fully Compatible | ||||
| Operating System: | ALL | ||||
| Sprint: | Repl 2017-05-29, Repl 2017-06-19 | ||||
| Participants: | |||||
| Linked BF Score: | 0 | ||||
| Description |
|
Some implementations of the audit function logCommandAuthzCheck() depend on a couple of generic Command functions (parseNs() and redactForLogging()). This introduces an implicit dependency on the 'commands' library where Command is defined, We have a cyclic dependency between the Command and logCommandAuthzCheck() because Command::checkAuthorization() calls logCommandAuthzCheck(). |
| Comments |
| Comment by Githook User [ 25/May/17 ] |
|
Author: {u'username': u'benety', u'name': u'Benety Goh', u'email': u'benety@mongodb.com'}Message: This import is no longer necessary now that the downstream changes to implementations of the |
| Comment by Githook User [ 25/May/17 ] |
|
Author: {u'username': u'benety', u'name': u'Benety Goh', u'email': u'benety@mongodb.com'}Message: |
| Comment by Githook User [ 25/May/17 ] |
|
Author: {u'username': u'benety', u'name': u'Benety Goh', u'email': u'benety@mongodb.com'}Message: |