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.

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

Avoid relay with same name overwrite, add get & delete rest api for relay #489

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

adibzter
Copy link

@adibzter adibzter commented Oct 8, 2021

Deleting relay using api will stop the session but the stats of the relay will remain available. Not really sure how to remove it completely.

@adibzter adibzter changed the title Avoid relay with same name overwrite Avoid relay with same name overwrite, add get & delete rest api for relay Oct 9, 2021
@adibzter
Copy link
Author

adibzter commented Oct 9, 2021

For the naming "delStream", I use the same naming with the existing "getStreams", "pullStream" etc. But I think it's proper to name it "delRelay". Just my opinion.

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.

1 participant