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.

Sign up for GitHub

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

Jump to bottom

Cache the bucket if the key is limited to one #362

Merged
merged 10 commits into from
Nov 23, 2022
Merged

Cache the bucket if the key is limited to one #362

merged 10 commits into from
Nov 23, 2022

Conversation

kkalinowski-reef
Copy link
Collaborator

  • checking whether there is a single bucket allowed for authorised key
  • when the name of the bucket is missing, inform user about the issue

ppolewicz and others added 8 commits May 3, 2022 22:43
@ppolewicz
WIP
630cb0a
@kkalinowski-reef
Merge branch 'master' into no_listBuckets
1366cd6 
* master: (149 commits)
  Clarification on the changelog
  Stats collector using context managers to hold the data
  Prepare release 1.18.0
  Apply code review feedback
  Fix syntax, formatting and comment in exception.py
  Do not run a the test with extra parameters on old apiver
  Add test of B2Api/Services initialization
  Rename parameter for consistency
  Fix raising CopySourceTooBig
  Clean up interpret_b2_errors and fix file lock exceptions
  Add is_file_lock_enabled parameter to Bucket.update()
  remove useless line
  More detailed download performance logs
  Add logging performance summary of parallel download threads
  Refactor stats collector to be usable for other threads
  Fix CopySizeTooBig exception
  Add download stats collector for parallel download writer thread
  Make streams loggable
  Refactor get_key test
  Fix B2Api.get_key() returning wrong key if `key_id` is incorrect
  ...
@kkalinowski-reef
API takes care of managing bucket cache
ab96414
@kkalinowski-reef
Tests ensuring that list bucket is not called if authorised for bucket
0c87dd4
@kkalinowski-reef
advanced.rst updated after codespell
159dc2f
@kkalinowski-reef
Changelog updated
c0f9534
@kkalinowski-reef
PR fixes
66bca2f 
- Moved bucket caching to a new function
- Fixed super() on all exceptions
@kkalinowski-reef
Lint fix
4e8278b
ppolewicz
ppolewicz reviewed Nov 22, 2022
View reviewed changes
Copy link
Collaborator

@ppolewicz ppolewicz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pep8 fix

@ppolewicz ppolewicz merged commit f7c1626 into Backblaze:master Nov 23, 2022
@ppolewicz ppolewicz deleted the no_listBuckets branch November 23, 2022 00:10
mjurbanski-reef added a commit that referenced this pull request Mar 29, 2024
@mjurbanski-reef
Merge pull request #362 from reef-technologies/move_internal_misc
0564f90 
Fix moved paths in docstrings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ppolewicz ppolewicz ppolewicz approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@kkalinowski-reef @ppolewicz