Skip to content
This repository has been archived by the owner on Mar 23, 2022. It is now read-only.

Add attributes to tunnel to store meta information. #3

Merged
merged 1 commit into from
Mar 17, 2020

Conversation

AndrienkoAleksandr
Copy link
Contributor

What does this PR do?

eclipse-che/che#16250

What issues does this PR fix or reference?

Add attributes to tunnel to store meta information.

Signed-off-by: Oleksandr Andriienko [email protected]

@skabashnyuk
Copy link
Contributor

skabashnyuk commented Mar 16, 2020

@AndrienkoAleksandr
Copy link
Contributor Author

how it is related to eclipse-che/che#16250 ?

We need to have ability store meta-data information in the tunnel. For issue how it is related to eclipse-che/che#16250 we need to store token.

@skabashnyuk
Copy link
Contributor

skabashnyuk commented Mar 17, 2020

  • Does this token have any lifetime or it's endless?
  • What will happen after application restart/upgrade?

@AndrienkoAleksandr
Copy link
Contributor Author

Does this token have any lifetime or it's endless?

@sleshchenko ?

@sleshchenko
Copy link
Member

@skabashnyuk I'm not sure about token's lifetime, but che-machine-exec uses it only during terminal initializing...

What will happen after application restart/upgrade?

Che-machine-exec loses all websocket connections and users have to provide new token once CloudShell UI is refreshed.

@AndrienkoAleksandr AndrienkoAleksandr merged commit 931966b into master Mar 17, 2020
@AndrienkoAleksandr AndrienkoAleksandr deleted the addAttributesToTheTunnel branch March 17, 2020 13:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants