René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"PEP8 and other issues","articleBody":"Please fix them both in the example project here and in the online tutorial. :)\r\n\r\n### PEP8\r\n- spaces around the `=` of keywords args - PEP8 forbids spaces there\r\n- inconsistent indentation (some function bodies are indented with 8 spaces)\r\n- local imports for no reason (`datetime` in `hello_there`) - not directly a PEP8 violation but local stdlib imports don't really make sense\r\n\r\n### Flask stuff\r\n- please consider removing the example with the hardcoded HTML. some people WILL think it's OK to do that even if you mention that usually templates are used. also, that hardcoded-html example is vulnerable to XSS.\r\n- `href=\"/static/site.css\"` - use `href=\"{{ url_for('static', filename='site.css')}}\"`\r\n- `message=\"Hello there, \" + name + \"!\",` - usually you would pass just the `name` to the template and then build the message in there. So I would change this, and also do the `strftime` call in the template (ideally someone would use e.g. flask-babel but that's out of the scope of a simple tutorial)\r\n\r\nNote: I'm one of the flask maintainers, and I can assure you that people WILL assume things are good  / ok / the way to go when part of a tutorial. I've seen my share of bad practices from code people posted in our IRC channel.","author":{"url":"https://github.com/ThiefMaster","@type":"Person","name":"ThiefMaster"},"datePublished":"2018-07-06T08:43:07.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":5},"url":"https://github.com/1/python-sample-vscode-flask-tutorial/issues/1"}