b2-sdk-python icon indicating copy to clipboard operation
b2-sdk-python copied to clipboard

List of things to consider for apiver v2

Open ppolewicz opened this issue 5 years ago • 6 comments

  • https://github.com/Backblaze/b2-sdk-python/pull/129
  • replace InMemoryAccountInfo with SqliteAccountInfo and a special :memory: file, see here
  • change all path arguments everywhere (internally too) to use pathlib instead of strings

ppolewicz avatar Jun 21 '20 20:06 ppolewicz

  • change the API of SyncReport
  • the API should not exit from the given ProgressListener. It should be used as a context manager by the user as described in the documentation.

mlech-reef avatar Oct 10 '20 15:10 mlech-reef

  • able to set the timeout per API call, not globally
  • better format of progress reporting in SyncReport. Maybe merge total and compared as it's only different when the source is local.

mlech-reef avatar Oct 15 '20 12:10 mlech-reef

  • remove the workaround of setting time

^ no longer applicable

mlech-reef avatar Oct 15 '20 12:10 mlech-reef

  • Deliver reef-technologies/b2-sdk-python#110

mlech-reef avatar Feb 11 '21 16:02 mlech-reef

change RawApi to RawHttpApi in v2

mpnowacki-reef avatar Apr 20 '21 14:04 mpnowacki-reef

Some of the suggestions above did not make it to apiver v2 (but were not discarded):

  • replace InMemoryAccountInfo with SqliteAccountInfo and a special :memory: file, see here - this can be done in backward compatible way, and therefore does not require an apiver bump up
  • change all path arguments everywhere (internally too) to use pathlib instead of strings
  • the API should not exit from the given ProgressListener. It should be used as a context manager by the user as described in the documentation.
  • able to set the timeout per API call, not globally

mpnowacki-reef avatar Jun 24 '21 08:06 mpnowacki-reef