opencti
opencti copied to clipboard
[backend] check user access rights before merging
Proposed changes
- We now check that the user has access to all related relations before merging
Related issues
- #7043
Checklist
- [x] I consider the submitted work as finished
- [x] I tested the code for its functionality
- [ ] I wrote test cases for the relevant uses case (coverage and e2e)
- [ ] I added/update the relevant documentation (either on github or on notion)
- [ ] Where necessary I refactored code to improve the overall quality
Further comments
Codecov Report
Attention: Patch coverage is 65.21739% with 8 lines in your changes missing coverage. Please review.
Project coverage is 67.56%. Comparing base (
7b1f42b) to head (124548a). Report is 4 commits behind head on master.
| Files | Patch % | Lines |
|---|---|---|
| ...latform/opencti-graphql/src/database/middleware.js | 75.00% | 5 Missing :warning: |
| ...latform/opencti-graphql/src/manager/taskManager.js | 0.00% | 3 Missing :warning: |
Additional details and impacted files
@@ Coverage Diff @@
## master #6989 +/- ##
=======================================
Coverage 67.56% 67.56%
=======================================
Files 567 567
Lines 69903 69917 +14
Branches 5937 5939 +2
=======================================
+ Hits 47228 47239 +11
- Misses 22675 22678 +3
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.