search

jeffwidman / bitbucket-issue-migration

A small script for migrating repo issues from Bitbucket to GitHub
GNU General Public License v3.0
316 stars 95 forks source link

Unicode handling improvements and error reportin #18

Closed nskvortsov closed 8 years ago

nskvortsov commented 9 years ago

I faced different issues while trying to move stuff from bitbucket.

  1. Unicode symbols in usernames and comments caused some nasty UnicodeDecodeError-s. So changed stuff to be mostly unicode objects
  2. GitHub http errors can have meaningful body (e.g., rate limits hit). Added a clause to print them.

Python is not my best language, so please review.

rashivkp commented 9 years ago

@nskvortsov Thank you, this is useful. anyway I stuck at here.

Response was [ {"message":"You have triggered an abuse detection mechanism and have been temporarily blocked from content creation. Please retry your request again later.","documentation_url":"https://developer.github.com/v3#abuse-rate-limits"} ]

nskvortsov commented 9 years ago

You've triggered github's abuse prevention. This can be caused by large number of requests produced by migration script.

Please, contact github support, explain what were you doing and ask to whitelist your account. 01.03.2015 14:58 пользователь "Mohamed Rashid" notifications@github.com написал:

@nskvortsov https://github.com/nskvortsov Thank you, this is useful. anyway I stuck at here.

Response was [ {"message":"You have triggered an abuse detection mechanism and have been temporarily blocked from content creation. Please retry your request again later.","documentation_url":" https://developer.github.com/v3#abuse-rate-limits"} ]

— Reply to this email directly or view it on GitHub https://github.com/vbabiy/bitbucket_issue_migration/pull/18#issuecomment-76587085 .

ingomueller-net commented 9 years ago

Works like a charm :) Thanks! @nskvortsov, it would be great if you could accept the PR.

nskvortsov commented 9 years ago

@ingomueller-net you are welcome. Unfortunately, I do not have permissions to accept my own PR :) @vbabiy would you mind doing it?

jeffwidman commented 8 years ago

Thanks @nskvortsov for the PR. New project maintainer here, so still coming up to speed.

25 did a fairly massive overhaul that switched to the new Github Import API, so the rate limiting errors shouldn't be an issue anymore. It also made some changes to the unicode handling, so I think that's fixed as well.

If not, please rebase and submit a new PR. Ideally the unicode and rate-limiting issues are kept as separate PR's as they're easier to track/discuss/merge that way.

nskvortsov commented 8 years ago

@jeffwidman Glad to hear that the project will be maintained from now on! Good luck, @jeffwidman !