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

Add option to create torrents #517

Open
GH0STDATA opened this issue Feb 19, 2023 · 6 comments
Open

Add option to create torrents #517

GH0STDATA opened this issue Feb 19, 2023 · 6 comments

Comments

@GH0STDATA
Copy link

GH0STDATA commented Feb 19, 2023

Hey @johman10, hope you're having a good weekend.

It looks like the official Flood UI repo has the ability to create torrents directly in the web UI, but it seems to be removed or not implemented in this version.

jesec/flood#50

Will this be added in the future?

@johman10
Copy link
Owner

It seems like the package used for this would be: https://www.npmjs.com/package/create-torrent. It states it can be used in the browser, so this might be possible to do in this project too. I can consider it for the future. :)

Thank you for the feature request. :D

@GH0STDATA
Copy link
Author

Sounds good, thank you for responding and considering adding this feature!

It looks like there are some other major differences between your Flood clone and the official repo. One of them being that the official repo seems to have file browsing capabilities built into the UI, which is one of the couple of reasons why creating torrents is possible.

@johman10
Copy link
Owner

johman10 commented Feb 22, 2023

@GH0STDATA I would like to clarify that this project is in no way a clone of Flood. It's an independent project based on Flood UI jfurrow (with only rtorrent support). Both Jesec and me were working on a recreation of the project without knowing about each others efforts. When it was time for me to release I found Jesec's work and decided there is no pain in having both in this world. This means there is 0 technical overlap between the two projects.

Since Flood is build with a NodeJS backend it requires it's own running server to serve it's content. Flood for Transmission is made to just support Transmission and depends on the Transmission server to serve the files as simple HTML/JS/CSS. While that makes Flood for Transmission more lightweight, it does limit the capabilities (file browsing being one of them, because a browser can't just read the whole file system).

Because of all these things Flood for Transmission and Flood will never be on par when it comes to features. But I hope that Flood for Transmission can be a good alternative to those who don't want to run yet another service on their host machine.

@GH0STDATA
Copy link
Author

GH0STDATA commented Feb 22, 2023

Gotcha, makes perfect sense. The reason why I said it was a clone was because its written in your own README file for this repository. 😆 That might need to be changed if that is not the case as you stated. I can definitely tell that there are differences, and both are great for different reasons. Thank you for your detailed explanation on the history of this repo as it's quite interesting!

@johman10
Copy link
Owner

@GH0STDATA fair point! I forgot that I wrote that! 😅 Might have to give the README some more love soon! 😄

@johman10
Copy link
Owner

johman10 commented Nov 18, 2023

@GH0STDATA as you can see from the PR linked above I made an attempt at this. But I'm afraid it doesn't work.

I think the main problem is that from the UI you should be able to select a path to a folder that is part of the file system of the Transmission host. But since Transmission doesn't expose files, nor an API, there is no real way to load those files into the torrent without some implemetation on the backend (which is simply Transmission). Because of this I don't think this is possible with the current architecture.

However, I'm not an expert on this. So if you know anything I don't about the expected behaviour and how to implement it, please let me know here and I will have another look based on your input.

Please note that the current PR does some kind of file upload solution which was a desperate attempt to make it work, but that doesn't work because it means Transmission can't seed the file.

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

No branches or pull requests

2 participants