filepath, filename gets correctly set in send_message
Also updated the names of the parameters so that they don't get
overwritten by the function body - this reduces the amount of state one
has to keep track of when reading and/or updating the function.
Describe the pull request:
[x] Bug fix
[ ] Functional change
[ ] New feature
[ ] Code cleanup
[ ] Build system change
[ ] Documentation change
[ ] Language translation
Changes made:
notifications.py - fixed bug according to above
Steps to reproduce the original issue
Checkout 1f9d197 and make sure that all docker images are built against that revision
Set log = console in private/lega/conf.ini
run docker-compose up
run make upload in a different terminal inside the docker/test directory
The (relevant) log output from docker-compose will be:
inbox | [lega.notifications][ INFO ] (L41) Using chroot isolation
inbox | [lega.notifications][ INFO ] (L73) User ega-box-999 uploaded /file.txt.c4ga
inbox | [lega.utils.checksum][ERROR ] (L43) Unable to calculate checksum: IsADirectoryError(21, 'Is a directory')
inbox | Received disconnect from 192.168.240.1 port 54852:11: disconnected by user
inbox | Disconnected from user ega-box-999 192.168.240.1 port 54852
filepath, filename gets correctly set in send_message
Also updated the names of the parameters so that they don't get overwritten by the function body - this reduces the amount of state one has to keep track of when reading and/or updating the function.
Describe the pull request:
Changes made:
notifications.py- fixed bug according to aboveSteps to reproduce the original issue
1f9d197and make sure that all docker images are built against that revisionlog = consoleinprivate/lega/conf.inidocker-compose upmake uploadin a different terminal inside thedocker/testdirectoryThe (relevant) log output from docker-compose will be: