{"action":"created","issue":{"url":"https://api.github.com/repos/h5bp/server-configs-nginx/issues/95","labels_url":"https://api.github.com/repos/h5bp/server-configs-nginx/issues/95/labels{/name}","comments_url":"https://api.github.com/repos/h5bp/server-configs-nginx/issues/95/comments","events_url":"https://api.github.com/repos/h5bp/server-configs-nginx/issues/95/events","html_url":"https://github.com/h5bp/server-configs-nginx/issues/95","id":67867181,"number":95,"title":"Add \".conf\" Extension To Site Config Files","user":{"login":"quantumpacket","id":8516784,"avatar_url":"https://avatars.githubusercontent.com/u/8516784?v=3","gravatar_id":"","url":"https://api.github.com/users/quantumpacket","html_url":"https://github.com/quantumpacket","followers_url":"https://api.github.com/users/quantumpacket/followers","following_url":"https://api.github.com/users/quantumpacket/following{/other_user}","gists_url":"https://api.github.com/users/quantumpacket/gists{/gist_id}","starred_url":"https://api.github.com/users/quantumpacket/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/quantumpacket/subscriptions","organizations_url":"https://api.github.com/users/quantumpacket/orgs","repos_url":"https://api.github.com/users/quantumpacket/repos","events_url":"https://api.github.com/users/quantumpacket/events{/privacy}","received_events_url":"https://api.github.com/users/quantumpacket/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-04-12T03:39:41Z","updated_at":"2015-05-19T18:11:01Z","closed_at":null,"body":"Currently the sample site files and the docs say:\r\n\r\n>\r\nDefine host definitions here.\r\nIt'd be a good thing if you keep your hosts indexed by domain name, eg:\r\n>\r\n```\r\nexample.com (handles traffic from both www.example.com and example.com)\r\nfoobar.com (as above)\r\ntest.foobar.com (handles traffic from both www.test.foobar.com and test.foobar.com)\r\n```\r\n\r\nI agree with that using the domain name in the filename. However, since these files are Nginx config files they should retain that distinction and have a `.conf` extension. Not only for semantic purposes, but also to ensure IDEs will correctly parse the file as the intended filetype. I'm using PhpStorm with Nginx support and my config files are highlighted and checked for errors, but opening a `.com` file will not work.\r\n\r\nSince there are essentially countless TLDs that can be used these days. Not enforcing a `.conf` extension will cause a lot of files to be parsed wrongly by IDEs. Take for example the TLD `.py` for Paraguay. Do we really want a dev's IDE to think `example.py` is a Python file?\r\n\r\nMy proposal is to add the appropriate extension to the sample files and update the documentation. Thoughts?\r\n"},"comment":{"url":"https://api.github.com/repos/h5bp/server-configs-nginx/issues/comments/103619014","html_url":"https://github.com/h5bp/server-configs-nginx/issues/95#issuecomment-103619014","issue_url":"https://api.github.com/repos/h5bp/server-configs-nginx/issues/95","id":103619014,"user":{"login":"tchalvak","id":23353,"avatar_url":"https://avatars.githubusercontent.com/u/23353?v=3","gravatar_id":"","url":"https://api.github.com/users/tchalvak","html_url":"https://github.com/tchalvak","followers_url":"https://api.github.com/users/tchalvak/followers","following_url":"https://api.github.com/users/tchalvak/following{/other_user}","gists_url":"https://api.github.com/users/tchalvak/gists{/gist_id}","starred_url":"https://api.github.com/users/tchalvak/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tchalvak/subscriptions","organizations_url":"https://api.github.com/users/tchalvak/orgs","repos_url":"https://api.github.com/users/tchalvak/repos","events_url":"https://api.github.com/users/tchalvak/events{/privacy}","received_events_url":"https://api.github.com/users/tchalvak/received_events","type":"User","site_admin":false},"created_at":"2015-05-19T18:11:01Z","updated_at":"2015-05-19T18:11:01Z","body":"Apache 2.4 switched over to requiring .conf at some point in the last few years.  I think it's not too important, but .conf is a good default."}}