Skip to content

fix(session): reuse eventstore when getting session through nats#1029

Open
nabil-mansouri wants to merge 3 commits into
masterfrom
fix-session
Open

fix(session): reuse eventstore when getting session through nats#1029
nabil-mansouri wants to merge 3 commits into
masterfrom
fix-session

Conversation

@nabil-mansouri

Copy link
Copy Markdown
Member

Description

Ce fix permet de réutiliser les eventstore et éviter l'instanciation de pool pg

Fixes

(Enter here Jira or Redmine ticket(s) links)

Type of change

Please check options that are relevant.

  • Chore (PATCH)
  • Doc (PATCH)
  • Bug fix (PATCH)
  • New feature (MINOR)

Which packages changed?

Please check the name of the package you changed

  • admin
  • app-registry
  • archive
  • auth
  • cas
  • common
  • communication
  • conversation
  • directory
  • feeder
  • infra
  • portal
  • session
  • test
  • tests
  • timeline
  • workspace

Tests

  1. Describe here the tests you performed
  2. Step by step
  3. With expected results

Reminder

  • Security flaws

  • Performance impacts (think bulk !)

  • Unit tests were replayed

  • Unit tests were added and/or changed

  • I have updated the reminder for the version including my modifications

  • All done ! 😃

@nabil-mansouri
nabil-mansouri force-pushed the fix-session branch 3 times, most recently from 84966db to 408410f Compare May 28, 2026 16:56
@nabil-mansouri
nabil-mansouri force-pushed the fix-session branch 3 times, most recently from 72c645a to d20aa2f Compare May 28, 2026 17:06
@sonarqubecloud

Copy link
Copy Markdown

@juniorode juniorode left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

OK modulo les versions des dépendances

@benjaminperez

Copy link
Copy Markdown
Member

Oui, vu que la PR est sur master, il n'aurait fallu commit que le java.

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.

4 participants