Skip to content
This repository was archived by the owner on Dec 5, 2025. It is now read-only.

Conversation

@flavienSindou
Copy link
Contributor

@flavienSindou flavienSindou commented Dec 9, 2024

Proposed changes

  • upgrade SSE client version to add newline splitting patch

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • [] I wrote test cases for the relevant uses case
  • [] I added/update the relevant documentation (either on github or on notion)
  • [] Where necessary I refactored code to improve the overall quality

Related task with end to end error reproduction steps:

https://www.notion.so/filigran/SSE-client-encoding-1288fce17f2a802f9fecd23e09cde2d5?pvs=4#1288fce17f2a800296aad8bd48b31a0b

@flavienSindou flavienSindou self-assigned this Dec 9, 2024
@flavienSindou flavienSindou changed the title feat: update sse-cloent to handle newline character feat: update sse-client to handle newline character Dec 9, 2024
@SouadHadjiat SouadHadjiat added the filigran team use to identify PR from the Filigran team label Dec 10, 2024
@flavienSindou flavienSindou merged commit 711f3a9 into master Dec 17, 2024
6 checks passed
@flavienSindou flavienSindou deleted the connectors/issue/3061 branch December 17, 2024 09:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

filigran team use to identify PR from the Filigran team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants