[SERVER-33758] Remove logUserIds Created: 08/Mar/18 Updated: 29/Oct/23 Resolved: 26/Mar/18 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Logging |
| Affects Version/s: | 3.6.3 |
| Fix Version/s: | 3.7.4 |
| Type: | Improvement | Priority: | Trivial - P5 |
| Reporter: | Danny Hatcher (Inactive) | Assignee: | Benjamin Caimano (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Backwards Compatibility: | Minor Change | ||||||||||||||||
| Sprint: | Platforms 2018-03-26 | ||||||||||||||||
| Participants: | |||||||||||||||||
| Case: | (copied to CRM) | ||||||||||||||||
| Description |
|
logUserIds appears to no longer work in the latest versions of MongoDB. As this functionality is now included in Auditing instead, we should remove it. |
| Comments |
| Comment by Benjamin Caimano (Inactive) [ 26/Mar/18 ] |
|
Also cleaned up a mess of raw pointers that became unique pointers to become raw pointers again in the Encoder/Appender classes |
| Comment by Githook User [ 26/Mar/18 ] |
|
Author: {'email': 'ben.caimano@10gen.com', 'name': 'Ben Caimano'}Message: |
| Comment by Benjamin Caimano (Inactive) [ 26/Mar/18 ] |
|
I'm pushing the supplemental commits today. (Making sure that everything is still hunky-dory with yet another evergreen patch.) I'd say they'll be in by the time we have the sprint planning meeting. Want me to close it when I push them? |
| Comment by Githook User [ 21/Mar/18 ] |
|
Author: {'email': 'ben.caimano@10gen.com', 'name': 'Ben Caimano'}Message: Appenders and LogDomains now use unique ptrs internally and on |
| Comment by Githook User [ 16/Mar/18 ] |
|
Author: {'email': 'ben.caimano@10gen.com', 'name': 'Ben Caimano'}Message: Removed unused `--logUserIds` flag, consolidated logger encoding |
| Comment by Eric Milkie [ 08/Mar/18 ] |
|
The logUserIds parameter still exists in the codebase, though, and it appears to at least still have the ability to add a "user:" field. Something in the code logic must have broken. We also have a jstest for it; log_user_basic.js |
| Comment by Kelsey Schubert [ 08/Mar/18 ] |
|
It looks like this functionality was removed in 2.6 as part of |