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

[BUG] Cluster endpoint URL pattern validation for Server and Serverless domain in UX component before sending test connection request (Optional) #5706

Closed
xinruiba opened this issue Jan 17, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@xinruiba
Copy link
Member

xinruiba commented Jan 17, 2024

Describe the bug

When we try to create a datasource from "Dashboards Management" ---> "Data Sources" ---> "Create DataSource", we notice we if the datasource endpoint is start with "http" or "https", it will pass the endpoint validation.

Create this bug to:

  1. Check if there are more detailed endpoint validation we can do from front-end side.

To Reproduce
Steps to reproduce the behavior:

  1. Go to "Dashboards Management" ---> "Data Sources"
  2. Click on "Create data source connection"
  3. Fill in the form with DataSource Title. eg: "test"
  4. Fill in the form with DataSource Endpoint followed with a path name. eg: "https://opensearchendpoint/_dashboard"
  5. You will notice every endpoint start with "http" and "https" will be passed the validation

Expected behavior
Take a investigation to see if there are more endpoint format validations we can do.

In scope
This change should only in front-end side including following staff:

  1. Work with team and ux to figure out the expected endpoint pattern
  2. Endpoint pattern validation code change

Out of scope

  1. Endpoint correctness suggestions

Expected output

Detect endpoint format error inputed by our customer

@xinruiba xinruiba added bug Something isn't working untriaged labels Jan 17, 2024
@xinruiba xinruiba changed the title [BUG] Cluster endpoint URL pattern validation for Server and Serverless domain (Optional) [BUG] [UX] Cluster endpoint URL pattern validation for Server and Serverless domain before sending test connection request (Optional) Jan 17, 2024
@xinruiba xinruiba changed the title [BUG] [UX] Cluster endpoint URL pattern validation for Server and Serverless domain before sending test connection request (Optional) [BUG] Cluster endpoint URL pattern validation for Server and Serverless domain in UX component before sending test connection request (Optional) Jan 17, 2024
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

No branches or pull requests

2 participants