Skip to content

Navigation Menu

Appearance settings

View all features
- BY COMPANY SIZE
  Enterprises
  Small and medium teams
  Startups
  Nonprofits
- BY USE CASE
  App Modernization
  DevSecOps
  DevOps
  CI/CD
  View all use cases
- BY INDUSTRY
  Healthcare
  Financial services
  Manufacturing
  Government
  View all industries
View all solutions
- EXPLORE BY TOPIC
  AI
  Software Development
  DevOps
  Security
  View all topics
- EXPLORE BY TYPE
  Customer stories
  Events & webinars
  Ebooks & reports
  Business insights
  GitHub Skills
- SUPPORT & SERVICES
  Documentation
  Customer support
  Community forum
  Trust center
  Partners
View all resources
- COMMUNITY
  GitHub SponsorsFund open source developers
- PROGRAMS
  Security Lab
  Maintainer Community
  Accelerator
  GitHub Stars
  Archive Program
- REPOSITORIES
  Topics
  Trending
  Collections
- ENTERPRISE SOLUTIONS
  Enterprise platformAI-powered developer platform
- AVAILABLE ADD-ONS
  GitHub Advanced SecurityEnterprise-grade security features
  Copilot for BusinessEnterprise-grade AI features
  Premium SupportEnterprise-grade 24/7 support
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

Appearance settings

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

dfir-iris / iris-web Public

Notifications You must be signed in to change notification settings
Fork 288
Star 1.5k

Code
Issues 354
Pull requests 29
Discussions
Actions
Projects
Security and quality 5
Insights

Additional navigation options

Code
Issues
Pull requests
Discussions
Actions
Projects
Security and quality
Insights

Api v2 create object comment#914

Merged

whikernel merged 58 commits intodevelopdfir-iris/iris-web:developfrom

api_v2_create_object_commentdfir-iris/iris-web:api_v2_create_object_commentCopy head branch name to clipboard

Sep 29, 2025

Conversation Commits58 (58)Checks Files changed

Merged

Api v2 create object comment#914
whikernel merged 58 commits intodevelopfrom
api_v2_create_object_comment

Commits

Commits on Sep 17, 2025

One import per line
c8y3
committed
Started implementation of POST /api/v2/alerts/{object_identifier}/comments
c8y3
committed
POST /api/v2/alerts/{object_identifier}/comments should set comment_text
c8y3
committed
POST /api/v2/alerts/{object_identifier}/comments should set comment_alert_id
c8y3
committed
POST /api/v2/alerts/{object_identifier}/comments should set comment_user_id
c8y3
committed
POST /api/v2/alerts/{object_identifier}/comments should add an history entry on the alert
c8y3
committed
Factored some testing utilities for modules and hooks
c8y3
committed
POST /api/v2/alerts/{object_identifier}/comments should send a module hook
c8y3
committed
POST /api/v2/alerts/{object_identifier}/comments should send track activity
c8y3
committed
POST /api/v2/alerts/{object_identifier}/comments should return 404 when alert is not found
c8y3
committed
POST /api/v2/alerts/{object_identifier}/comments should return 400 when field has incorrect type
c8y3
committed
Started POST /api/v2/assets/{object_identifier}/comments
c8y3
committed
Factored code of alerts somewhat
c8y3
committed
POST /api/v2/assets/{object_identifier}/comments sets comment's text
c8y3
committed
POST /api/v2/assets/{object_identifier}/comments implemented 400 and 404
c8y3
committed
GET /api/v2/assets/{object_identifier}/comments should return 404, rather than 403 when access to asset is forbidden (so as to not leak any info about the existence of the asset)
c8y3
committed
Implementation of GET /api/v2/evidences/{object_identifier}/comments
c8y3
committed
Removed import of iris_current_user in business/comments (should not be imported from the business layer)
c8y3
committed
Use objects instead of identifiers
c8y3
committed
Fixed incorrect argument passed
c8y3
committed
Code cleanup
c8y3
committed
Code cleanup
c8y3
committed
Removed seemingly unused method
c8y3
committed
Removed unused imports
c8y3
committed
Moved code down into the persistence layer
c8y3
committed
Changed back type to int (Ideally, it should be a list of CaseAccessLevel
c8y3
committed
Removed unused parameter
c8y3
committed
Moved checks up in API layer
c8y3
committed
Used logger
c8y3
committed
More precise return type
c8y3
committed
Create comments is allowed only with write case access
c8y3
committed
Create comments is allowed only with write case access
c8y3
committed
POST /api/v2/iocs/{object_identifier}/comments
c8y3
committed
POST /api/v2/notes/{object_identifier}/comments
c8y3
committed
Deleting a case should delete associated comment
c8y3
committed
Fixed missing argument when calling function
c8y3
committed
Fixed deletion of case when there are comments on assets
c8y3
committed
Deleting a case, does not incorrectly delete comments on iocs in another case
c8y3
committed
Fixed deletion of case with evidence comments
c8y3
committed
Fixed deletion of case with notes comments
c8y3
committed
POST /api/v2/tasks/{object_identifier}/comments
c8y3
committed
Delete tasks comments when deleting a case
c8y3
committed
Factored code somewhat
c8y3
committed
commit seems to be necessary
c8y3
committed
POST /api/v2/events/{object_identifier}/comments
c8y3
committed
Deletion of case with events comments does not fail
c8y3
committed
Fixed some ruff warnings
c8y3
committed
Moved out definition of tables related to Comments in their own models file
c8y3
committed
After test generate log file and upload as an artifact
c8y3
committed
Deprecated previous endpoints to create comments
c8y3
committed
Forgot to redirect file
c8y3
committed
Renamed CI artifact so that it doesn't conflict
c8y3
committed
Clean: more precise return type
c8y3
committed
Avoid unnecessary iteration
c8y3
committed
Started to move access_control method out of namespace iris_engine into business layer
c8y3
committed
Moved function ac_flag_match_mask out of iris_engine namespace down into models
c8y3
committed
Moved functions out of iris_engine.access_control.utils into the business layer
c8y3
committed
Split code correctly between business and persistence layer
c8y3
committed

Footer

© 2026 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Community
Docs
Contact

You can’t perform that action at this time.