opencti icon indicating copy to clipboard operation
opencti copied to clipboard

[backend] check user access rights before merging

Open JeremyCloarec opened this issue 1 year ago • 1 comments

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

JeremyCloarec avatar May 16 '24 09:05 JeremyCloarec

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.

codecov[bot] avatar May 16 '24 09:05 codecov[bot]