social media sample application

[tmcallaghan]

Social media application for sample usecases.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[cod-all]

Architecture is clean and the DocumentDB integration is solid. A few small things:

1. login_required is duplicated in routes/users.py, routes/posts.py, and routes/search.py. Suggest extracting it into a shared utilities file.

2. routes/posts.py has the following -

for post in posts:
        author = db.users.find_one({'_id': post['author_id']})
        post['author'] = author

So if an author has 100 posts, it's going to do 100 findOne() operations. Should this be changed to a batch lookup?

3. users.py does a regex search - 'username': {'$regex': query, '$options': 'i'}

Which means if I type in .* , I can essentially get all users. Should there be a validation or escape added to avoid this?

[cod-all]

Changes look great. 2 other items to fix before merging, as these could cause issues with deployment.

1. The readme says "The application runs in debug mode by default when using python app.py ". But app.py has this:

debug = os.getenv('FLASK_DEBUG', 'false').strip().lower() in ('1', 'true', 'yes', 'on') 

Debug is off by default.

2. In the readme, "4. Run the Application" - you give two ways to run - using flask or python.  RUnning it via python will read the FLASK_PORT variable (3000). But running via flask doesn't - it will default to port 5000. So if launched via flask run, the localhost URL in the readme won't work.