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

Stockage des media dans un cloud S3 #250

Merged
merged 39 commits into from
Feb 22, 2021
Merged

Stockage des media dans un cloud S3 #250

merged 39 commits into from
Feb 22, 2021

Conversation

amandine-sahl
Copy link
Contributor

cf #248

@jbdesbas
Copy link
Contributor

Bonjour @amandine-sahl
Très intéressant comme approche, merci ! J'ai corrigé deux trucs dans #248 (et changer la cible de la PR vers cette branche)

@amandine-sahl
Copy link
Contributor Author

La réécriture du code pour utiliser le filemanager et media repository est bien avancé. Il reste juste une mention au stokage s3 en dehors du filemanager dans le repository que je ne vois pas comment déporter.

Par contre je n'ai pas tester le stockage s3. Je laisse @jbdesbas le faire.

@jbdesbas
Copy link
Contributor

Parfait merci, OK pour moi #254

Pour précision : une petite particularité c'est que, lors de l'utilisation du stockage S3, c'est bien le chemin (pas l'url) qui est stocké dans la base (car le media se gère bien comme un fichier), MAIS c'est l'url (construite à partir du chemin et de la config S3_PUBLIC_URL) qui est renvoyée par l'API publique (le chemin est ajouté si forcePath=True est rajouté à la requête (seulement utile pour l'utilisation de l'API dans Taxhub, pour l'édition de media).

@jbdesbas jbdesbas mentioned this pull request Nov 30, 2020
@jbdesbas
Copy link
Contributor

jbdesbas commented Dec 1, 2020

L'utilisation du stockage S3 nécessite la lib python boto3, faut-il l'inclure dans le requirements.txt où laisser l'utilisateur l'installer s'il désire utiliser le S3 ?
L'installation de boto3 pose un conflit avec pypnuuserhub sur la version de requests, j'ai toutefois pu utiliser la 2.25 sans constater de dysfonctionnements.

ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
pypnusershub 1.4.4 requires requests==2.21.0, but you have requests 2.25.0 which is incompatible.

@amandine-sahl amandine-sahl changed the base branch from develop to dev_merge_pull February 22, 2021 09:11
@amandine-sahl amandine-sahl merged commit 089dd3b into dev_merge_pull Feb 22, 2021
@amandine-sahl amandine-sahl mentioned this pull request Mar 9, 2021
@amandine-sahl amandine-sahl deleted the s3 branch June 22, 2021 14:21
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.

3 participants