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

Upload to nyaa.si, anidex & TokyoTosho [done] #1633

Merged
merged 68 commits into from
Jan 3, 2018
Merged

Upload to nyaa.si, anidex & TokyoTosho [done] #1633

merged 68 commits into from
Jan 3, 2018

Conversation

Kiloutre
Copy link
Contributor

@Kiloutre Kiloutre commented Oct 4, 2017

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 40.041% when pulling 40595bb on upload-multiple into 51d3520 on dev.

@Kiloutre Kiloutre changed the title [WIP] [WIP] Upload to nyaa.si, anidex & TokyoTosho Oct 4, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 40.041% when pulling f469fd1 on upload-multiple into 51d3520 on dev.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 40.041% when pulling 9ec4bc5 on upload-multiple into 5edfc9f on dev.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 39.713% when pulling 35871e8 on upload-multiple into 2044a3c on dev.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 39.713% when pulling 3f6e32f on upload-multiple into 2044a3c on dev.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 39.713% when pulling ec9bfac on upload-multiple into 2044a3c on dev.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 39.713% when pulling ec9bfac on upload-multiple into 2044a3c on dev.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 39.713% when pulling ec9bfac on upload-multiple into 2044a3c on dev.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 38.816% when pulling f3d7400 on upload-multiple into 6f47f1b on dev.

@Kiloutre
Copy link
Contributor Author

Kiloutre commented Nov 3, 2017

I'm getting a 404 error when trying to upload to other websites, the torrent is uploaded & torrent file generated but /upload/status/:id gives me a 404
I also get this error message in the console
[GIN-debug] [WARNING] Headers were already written. Wanted to override status code 302 with 500

Torrent generation also doesn't seem to work during a regular download.go call, it says it's generating but it's seemingly not

@akuma06
Copy link
Member

akuma06 commented Nov 5, 2017

Ok I will look into it when I have time :)

Support torrent file creation when upload without scraping
* Fix possible bug where no one is seeding a torrent yet (first time upload)
Add primary tracker in torrent file
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 38.586% when pulling 19f39ac on upload-multiple into 418dc81 on dev.

@akuma06
Copy link
Member

akuma06 commented Dec 21, 2017

So I've modified some parts to the torrent generation when uploading torrent file.
The download part works and the upload to anidex too.

Now we need to only fill in nyaasi/tosho form and request part.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.8%) to 39.607% when pulling d9701e3 on upload-multiple into 418dc81 on dev.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.8%) to 39.607% when pulling d9701e3 on upload-multiple into 418dc81 on dev.

@akuma06
Copy link
Member

akuma06 commented Dec 23, 2017

So I was looking to implement nyaasi upload but it seems that doing that would make less secure their credentials since nyaasi doesn't use an api key it seems.
I didn't find a documentation on how to use their api but I did find a python script (https://github.com/nyaadevs/nyaa/blob/master/utils/api_uploader_v2.py). It looks like you have to use a basic http authentification to use their api.
Beside the fact that asking the credentials from nyaasi to allow upload is not secure, I couldn't get their api to work with my Rest Client (same with their python file). It is always sending me this error:

{
  "errors": {
    "recaptcha": [
      "The response parameter is missing."
    ]
  }
}

If someone can tell me if their python script is up to date and how their api works, that would be awesome!

…d by URL (and not torrent file upload). We need to make the download url available without ddos protection.

Also TokyoTosho doesn't need a category select. Since it does have limited option, we can automatically convert our categories to the tokyotosho one.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.8%) to 39.607% when pulling b0d32ff on upload-multiple into 418dc81 on dev.

…ckage.

Added category conversion for nyaasi too.
Added tests on category conversion.
Is not tested though.
@coveralls
Copy link

Coverage Status

Coverage decreased (-3.7%) to 35.087% when pulling 3604dee on upload-multiple into 418dc81 on dev.

@akuma06 akuma06 changed the title [WIP] Upload to nyaa.si, anidex & TokyoTosho Upload to nyaa.si, anidex & TokyoTosho Dec 25, 2017
@akuma06 akuma06 changed the title Upload to nyaa.si, anidex & TokyoTosho Upload to nyaa.si, anidex & TokyoTosho [done] Dec 25, 2017
@coveralls
Copy link

Coverage Status

Coverage decreased (-3.7%) to 35.111% when pulling dbc7d66 on upload-multiple into 418dc81 on dev.

@coveralls
Copy link

Coverage Status

Coverage decreased (-3.7%) to 35.111% when pulling dbc7d66 on upload-multiple into 418dc81 on dev.

@ewhal ewhal merged commit d5c2c6b into dev Jan 3, 2018
@ewhal ewhal deleted the upload-multiple branch January 3, 2018 08:21
@ewhal
Copy link
Member

ewhal commented Jan 3, 2018

<3

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.

None yet

4 participants