Skip to content

Fix issue with diffbot where Synced source is displayed with/without …#884

Open
falak-asad wants to merge 2 commits intotechstory/knowledge-sync-feature-branchfrom
bug/126857-source-not-getting-updated
Open

Fix issue with diffbot where Synced source is displayed with/without …#884
falak-asad wants to merge 2 commits intotechstory/knowledge-sync-feature-branchfrom
bug/126857-source-not-getting-updated

Conversation

@falak-asad
Copy link
Contributor

@falak-asad falak-asad commented Feb 26, 2026

#Fixes

  • AB#126857

Changelog Section

  • Cognigy.AI

Changelog

  • Fix bug in diffbot knowledge connector

Success criteria

  • The sources are updated correctly by diffbot

How to test

  1. Create diffbot webpage source with this URL https://www.utctime.net
  2. Run sync. and it should update the source without changing the source name

Security

  • Possible injection vector
  • Authentication/Access controls touched
  • Sensitive Data could be exposed
  • XSS
  • Logging/Monitoring touched
  • Exchanges data with external systems
  • No security implications

Additional considerations

  • This PR impacts NLU
  • This PR might have performance implications
  • This PR changes an existing data model (and might affect existing legacy data)
    • Examples: adding, renaming, removing a field or changing the format or constraints of a field
  • This PR might affect indexing
    • Examples: adding a new param for model query from DB without creating a new index for this model

Documentation Considerations

No additional documentation required.

@graymalkin77
Copy link

graymalkin77 commented Feb 26, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

…uses new version of the Knowledge Connectors with old version of Cognigy AI
@tsafs
Copy link

tsafs commented Feb 27, 2026

@falak-asad I would like to test, but to make it easier, what do you think about always supplying the latest build of the extension that reviewers can simply import without building themselves?

Copy link

@tsafs tsafs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are files changed for the confluence connector, but changes are not described in the PR description nor test steps provided.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants