-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
git bot closing unfixed issues? #2284
Comments
#2222 is not an actual bug imho, if you are using an external id, as soon as infohash is obtained, that should be used to get it. Feature request to get it on par is something I'm open to reviewing if anyone is willing to work on it. Bugs/security issues should never be stale and closed, we have certain rules to prevent that. Happy to hear your thoughts |
That's the point - you can't get the infohash from the id without doing an "add", which is a: expensive and unnecessary if its previously been added, and b: I believe will fail if its previously been added. The only workaround I could find was to keep a separate structure in the caller that maps external id to infohash. The usecase that failed was going to a second webpage that had the same video on it. I'd have worked on it, but the code is pretty obscure without the comments (as discussed by Bruce in #2279) so its really hard to figure out how to fix something like this - that requires extra data structure in the library - without breaking something else that is making different assumptions. |
One could re-implement the full parsing (remote or not) as the add to the get. If someone is willing to PR, I'll review it. But I don't think maintainers will. |
Yes - probably, except with poorly documented code its easier to build a workaround than to fix the bug :-( which is what I did a long time ago. |
A simple rule that might alleviate the issue is only marking stale and closing issues that have a maintainer's comment as the most recent. Here's another example of an issue that never got a resolution #2245 |
If this package is not actively developed anymore you should mention it on the README, as per https://gist.github.com/cheerfulstoic/d107229326a01ff0f333a1d3476e068d |
It is actively mantained, but the few of us can’t be in top of everything. I’m sorry we’ve missed a few issues.
Feel free to bump bugs as much as needed and tag me personally.
… On 29/03/2022, at 11:06 PM, Banou ***@***.***> wrote:
If this package is not actively developed anymore you should mention it on the README, as per https://gist.github.com/cheerfulstoic/d107229326a01ff0f333a1d3476e068d
It gives the users transparency on the state of the project and doesn't make people waste their times on reporting issues that will just get closed by default.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.
|
Gonna close this and try my best to have eyes on everything :) |
It looks like git-bot is closing issues that are unfixed, and they do not appear to be re-openable by mere mortals, but considered by it "Stale" this makes me concerned that there may be many more problems that are still extant, but not showing up because the team working on this seems to be small, and the lack of documentation in the code makes it hard for causual bug finders to dip in and fix one problem without causing others ?
(The specific issue is #2222 , and I'm not suggesting its a high priority issue, but I bet it bites someone else trying to build something that uses webtorrent, also of relevance is #2279 by @Bruce-Hopkins about the lack of commenting).
The text was updated successfully, but these errors were encountered: