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

DB config all editable roles #407

Closed
Naseem77 opened this issue Sep 5, 2024 · 0 comments · Fixed by #419
Closed

DB config all editable roles #407

Naseem77 opened this issue Sep 5, 2024 · 0 comments · Fixed by #419
Assignees
Labels
bug Something isn't working

Comments

@Naseem77
Copy link
Contributor

Naseem77 commented Sep 5, 2024

  • Details:
    bug occurs when attempting to change certain numeric fields in the settings, including MAX_QUEUED_QUERIES, TIMEOUT, TIMEOUT_MAX, TIMEOUT_DEFAULT, RESULTSET_SIZE, QUERY_MEM_CAPACITY, VKEY_MAX_ENTITY_COUNT, and MAX_INFO_QUERIES. According to the documentation, these fields should only accept valid numeric values (numbers without any characters or special symbols). However, an error is triggered when trying to update any of these fields to a value that starts with a number but includes characters or letters afterward.
    This issue is reproducible through both the API and UI on the staging repository.

  • Steps to Reproduce:

1- Navigate to the settings page.
2- Attempt to update the (MAX_QUEUED_QUERIES, TIMEOUT, TIMEOUT_MAX, TIMEOUT_DEFAULT, RESULTSET_SIZE, - QUERY_MEM_CAPACITY, VKEY_MAX_ENTITY_COUNT, MAX_INFO_QUERIES) with a value that starts with a number but is followed by characters or special symbols (e.g., 100s, 1@, 50abc, 00-1, 0s, 01-@#).
3- Observe the error message.

Image

Image

Image

@Naseem77 Naseem77 added the bug Something isn't working label Sep 5, 2024
@Anchel123 Anchel123 linked a pull request Sep 8, 2024 that will close this issue
Anchel123 added a commit that referenced this issue Sep 9, 2024
Fix #407 allowed only numbers as configuration value
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants