Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Custom Properties Enum API accepts any value #18775

Closed
Prajwal214 opened this issue Nov 25, 2024 · 0 comments · Fixed by #18855
Closed

Custom Properties Enum API accepts any value #18775

Prajwal214 opened this issue Nov 25, 2024 · 0 comments · Fixed by #18855
Assignees
Labels
backend bug Something isn't working customer

Comments

@Prajwal214
Copy link
Contributor

Affected module
Does it impact the UI, backend or Ingestion Framework?
-- Backend

Describe the bug
A clear and concise description of what the bug is.
-- Using the API for update custom properties enum does not any check before add values, it response correct when wrong values are introduced.

For example my custom porperty:
testAutomationHubMultiselect
["multiselectFirst","multiselectSecond"]
Multi Select: No

My custom property is not multiselect and it is supposed just to accept 2 specific values, however as you can see in the attached images can be update throw the API any value and any amount of value that can manage it.

To Reproduce

Screenshots or steps to reproduce
image

image

Expected behavior
A clear and concise description of what you expected to happen.

Version:

  • OS: [e.g. iOS]
  • Python version:
  • OpenMetadata version: [e.g. 0.8] OM v1.5.11
  • OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[docker]==XYZ]

Additional context
Add any other context about the problem here.

@Prajwal214 Prajwal214 added bug Something isn't working backend customer labels Nov 25, 2024
mohityadav766 pushed a commit that referenced this issue Dec 2, 2024
* Minor : Fix #18775 validation on enum custom property

* fix tests

* Add tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend bug Something isn't working customer
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants