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.

Sign up for GitHub

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

Jump to bottom

Create configuration reports validation errors #3021

Open
erikcurrin-elastic opened this issue Dec 6, 2024 · 1 comment · Fixed by #3059
Open

Create configuration reports validation errors #3021

erikcurrin-elastic opened this issue Dec 6, 2024 · 1 comment · Fixed by #3059
Assignees
@artem-shelkovnikov
Labels
bug Something isn't working effort:low good first issue Good for newcomers priority:high

Comments

@erikcurrin-elastic
Copy link

erikcurrin-elastic commented Dec 6, 2024 •
edited
Loading

Bug Description

When I am creating a new connector, I enter the required field values, I get a validation error for every field

To Reproduce

Steps to reproduce the behavior (using ServiceNow):

  1. In an 8.17.0 deployment, +New Connector
  2. Select ServiceNow as the Connector
  3. Click Generate Configuration
  4. Click Next
  5. Click Edit Configuration
  6. Fill in the information to make the connection
  7. Click Save

This also happens for SharePoint

Expected behavior

No errors

Actual behavior

Connection Error
Field validation errors for all the fields

Screenshots

Image

Environment

  • ES Version 8.17.0
@erikcurrin-elastic erikcurrin-elastic changed the title Create configuration reports validation error for username/password empty Create configuration reports validation errors Dec 6, 2024
@artem-shelkovnikov artem-shelkovnikov self-assigned this Dec 25, 2024
@artem-shelkovnikov artem-shelkovnikov mentioned this issue Dec 25, 2024
Merged
8 tasks
@artem-shelkovnikov
Copy link
Member

Reopening after testing: I've fixed part of it, but there's a problem somewhere in Kibana that creates native connectors in already "Configured" state.

To fix it, we need to have connector in "needs_configuration" step until user actually saves configuration.

@artem-shelkovnikov artem-shelkovnikov added the good first issue Good for newcomers label Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@artem-shelkovnikov artem-shelkovnikov

Labels
bug Something isn't working effort:low good first issue Good for newcomers priority:high
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Set connector status to CONNECTED after successful validation and ping elastic/connectors
3 participants
@seanstory @artem-shelkovnikov @erikcurrin-elastic