udacity / cd0157-Server-Deployment-and-Containerization

Server Deployment and Configuration project. This was formerly the nd0044 course 4 project.
74 stars 2.88k forks source link

Add str() around content length calculation #20

Closed Meicroft closed 3 years ago

Meicroft commented 4 years ago

This fixers an issue that took a ferw hours to trobuleshoot. Apparently the headers dict really, really needs to str() the response_body's len() calculation.

horvatha commented 3 years ago

Please merge this pull request. It works and the original version cases a lot of trouble. I've checked the knowledge base and the chat rooms, there is a lot of message about it. Thanks.

SudKul commented 3 years ago

Thanks, @Meicroft and @horvatha We have used your suggested updates, along with a few other modifications in this commit. We appreciate your inputs. Happy learning Udacity