diff --git a/attachment_s3/README.rst b/attachment_s3/README.rst index 1ebef0c..0d03699 100644 --- a/attachment_s3/README.rst +++ b/attachment_s3/README.rst @@ -18,7 +18,7 @@ Configure accesses with environment variables: * ``AWS_REGION`` (required if using AWS services) * ``AWS_ACCESS_KEY_ID`` * ``AWS_SECRET_ACCESS_KEY`` -* ``AWS_BUCKETNAME`` +* ``AWS_BUCKETNAME`` (optional {db} placeholder) Read-only mode: @@ -34,6 +34,22 @@ This addon must be added in the server wide addons with (``--load`` option): ``--load=web,attachment_s3`` +The System Parameter ``ir_attachment.storage.force.database`` can be customized to +force storage of files in the database. See the documentation of the module +``base_attachment_object_storage``. + +Multi-tenancy +------------- + +Use the `{db}` placeholder to handle multi-tenancy. + +On instances that hold multiple databases, it's preferable to have one bucket per database. + +To handle this, you can insert the `{db}` placeholder in your bucket name variable ``AWS_BUCKETNAME``. +It will be replaced by the database name. +This will give you a unique bucketname per database. + + Limitations ----------- diff --git a/attachment_s3/models/ir_attachment.py b/attachment_s3/models/ir_attachment.py index f3947fa..afaea94 100644 --- a/attachment_s3/models/ir_attachment.py +++ b/attachment_s3/models/ir_attachment.py @@ -51,6 +51,8 @@ class IrAttachment(models.Model): access_key = os.environ.get('AWS_ACCESS_KEY_ID') secret_key = os.environ.get('AWS_SECRET_ACCESS_KEY') bucket_name = name or os.environ.get('AWS_BUCKETNAME') + # replaces {db} by the database name to handle multi-tenancy + bucket_name.format(db=self.env.cr.dbname) params = { 'aws_access_key_id': access_key,