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

Sessions are not removed if removed by fabric #1626

Open
andrewpmartinez opened this issue Aug 24, 2020 · 0 comments
Open

Sessions are not removed if removed by fabric #1626

andrewpmartinez opened this issue Aug 24, 2020 · 0 comments
Assignees
@andrewpmartinez
Labels
bug Something isn't working controller Issue related to the controller

Comments

@andrewpmartinez
Copy link
Member

Experienced
Having edge routers go down will remove active sessions that have no possible route at the fabric level. Querying for the sessions via the Edge API will still show the session active. Clients of that session will not know it is invalid when querying the Edge API.

Expected
If the fabric removes a session, that is reflected in the Ede API

Reproduction

  1. Create a service w/ only 1 edge router that can handle it
  2. Connect everything (edge router, sdk, controller)
  3. Stop the edge router
  4. Use ziti-fabric list session and GET /sessions to compare.
@andrewpmartinez andrewpmartinez self-assigned this Aug 24, 2020
andrewpmartinez referenced this issue in openziti/fabric Aug 24, 2020
@andrewpmartinez
adds session remove handler for openziti/edge#285
f7ff88b
andrewpmartinez referenced this issue in openziti/fabric Aug 25, 2020
@andrewpmartinez
Merge pull request #120 from openziti/expose.session.remove
39a9757 
adds session remove handler for openziti/edge#285
@andrewpmartinez andrewpmartinez added bug Something isn't working controller Issue related to the controller labels Apr 21, 2021
plorenz referenced this issue Sep 29, 2023
@andrewpmartinez
adds session remove handler for openziti/edge#285
09e3aff
plorenz referenced this issue Sep 29, 2023
@andrewpmartinez
Merge pull request openziti/fabric#120 from openziti/expose.session.r…
3afcb6c 
…emove

adds session remove handler for openziti/edge#285
@plorenz plorenz transferred this issue from openziti/edge Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andrewpmartinez andrewpmartinez

Labels
bug Something isn't working controller Issue related to the controller
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@andrewpmartinez