Skip to content

Comments

Add ENV PMA_UPLOADDIR and PMA_SAVEDIR ENVs#384

Merged
williamdes merged 4 commits intophpmyadmin:masterfrom
chrisproud:dir-envvars
Nov 25, 2022
Merged

Add ENV PMA_UPLOADDIR and PMA_SAVEDIR ENVs#384
williamdes merged 4 commits intophpmyadmin:masterfrom
chrisproud:dir-envvars

Conversation

@chrisproud
Copy link

When hosting this image on an Azure hosted container instance, I can mount an Azure FileShare to a local path for uploads but I can't add custom file to the local volume to set the configuration.

This PR adds support setting the UploadDir and SaveDir through environment variables.

Co-authored-by: William Desportes <williamdes@wdes.fr>
@williamdes
Copy link
Member

@MauricioFauth @ibennetch If you have no objection I will merge this one soon and document it on the main repository

@chrisproud
Copy link
Author

@williamdes , anything I can do to help fix this broken test?

@williamdes
Copy link
Member

@williamdes , anything I can do to help fix this broken test?

Nevermind, I restarted the job. It's on my todo list I will write a test for this feature and merge it :)

@williamdes williamdes self-assigned this Oct 3, 2022
@williamdes williamdes changed the title [+] ENV PMA_UPLOADDIR and PMA_SAVEDIR Add ENV PMA_UPLOADDIR and PMA_SAVEDIR ENVs Nov 25, 2022
@williamdes williamdes merged commit 71d54ac into phpmyadmin:master Nov 25, 2022
williamdes added a commit that referenced this pull request Nov 25, 2022
@williamdes
Copy link
Member

Thank you for your contribution !
I merged it and wrote tests for it in:

williamdes added a commit to phpmyadmin/phpmyadmin that referenced this pull request Nov 25, 2022
Ref:
- phpmyadmin/docker#384
- phpmyadmin/docker#381

Signed-off-by: William Desportes <williamdes@wdes.fr>
@williamdes
Copy link
Member

Deployed on https://hub.docker.com/_/phpmyadmin ! 🚀

@chrisproud chrisproud deleted the dir-envvars branch November 28, 2022 08:14
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