opencti icon indicating copy to clipboard operation
opencti copied to clipboard

Mandatory field on External Reference is not correctly handle on the creation form

Open Lhorus6 opened this issue 1 year ago • 6 comments

Description

When I create an External Reference the form doesn't correctly display the information that the URL is mandatory

image

Environment

OCTI 6.3.4

Reproducible Steps

Steps to create the smallest reproducible scenario:

  1. Go on a Report
  2. Scroll down to add an external ref
  3. Create an Ext Ref -> fill in the "name" field but not the "URL" and try to create

Expected Output

Example with the name field:

image

Additional information

Maybe related, it's not noted as mandatory in the CSV mapper either:

image

Lhorus6 avatar Oct 04 '24 16:10 Lhorus6

Two corrective actions to handle:

  1. Mark the URL field as mandatory in the external reference creation form
  2. Mark the URL field as mandatory in the CSV mapper form

romain-filigran avatar Oct 07 '24 05:10 romain-filigran

URL is not mandatory, you have to put either an External Id, a URL or a File I think ?

Kedae avatar Oct 08 '24 07:10 Kedae

You right @Kedae.

romain-filigran avatar Oct 08 '24 07:10 romain-filigran

Oh ok, it's mandatory: Name AND (url OR external id OR file) ?

It may be complicated, but it has to be understandable.

Lhorus6 avatar Oct 08 '24 07:10 Lhorus6

May be we can try to address this matter with the "mandatory fields" work from community ? @frapuks @Jipegien @romain-filigran

Kedae avatar Oct 08 '24 07:10 Kedae

Currently, we have 3 fields touched by this issue :

  • Source name : always mandatory
  • External ID
  • Url

We need to have an external id OR an url.

The FBI PR is to display a star ater mandatory fields, and take account of the custom mandatory properties. But, we don't have customizable fields on External Ref.

I try to add a condition to set a external id as mandatory if url is empty : image But we can't use the same solution for the second field url because this is a cycling condition...

=> https://github.com/OpenCTI-Platform/opencti/compare/master...issue/8612

frapuks avatar Oct 11 '24 15:10 frapuks

Was it fixed with issue/8612 ?

SarahBocognano avatar Nov 06 '24 13:11 SarahBocognano

No, i create this branch only for trying a solution for a fast example. I don't work more on this issue.

frapuks avatar Nov 06 '24 14:11 frapuks