{"action":"created","issue":{"url":"https://api.github.com/repos/aperezdc/ngx-fancyindex/issues/27","labels_url":"https://api.github.com/repos/aperezdc/ngx-fancyindex/issues/27/labels{/name}","comments_url":"https://api.github.com/repos/aperezdc/ngx-fancyindex/issues/27/comments","events_url":"https://api.github.com/repos/aperezdc/ngx-fancyindex/issues/27/events","html_url":"https://github.com/aperezdc/ngx-fancyindex/issues/27","id":100443452,"number":27,"title":"Minor feature request: config option to exclude symbolic links","user":{"login":"DarkHelmet433","id":342419,"avatar_url":"https://avatars.githubusercontent.com/u/342419?v=3","gravatar_id":"","url":"https://api.github.com/users/DarkHelmet433","html_url":"https://github.com/DarkHelmet433","followers_url":"https://api.github.com/users/DarkHelmet433/followers","following_url":"https://api.github.com/users/DarkHelmet433/following{/other_user}","gists_url":"https://api.github.com/users/DarkHelmet433/gists{/gist_id}","starred_url":"https://api.github.com/users/DarkHelmet433/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DarkHelmet433/subscriptions","organizations_url":"https://api.github.com/users/DarkHelmet433/orgs","repos_url":"https://api.github.com/users/DarkHelmet433/repos","events_url":"https://api.github.com/users/DarkHelmet433/events{/privacy}","received_events_url":"https://api.github.com/users/DarkHelmet433/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/aperezdc/ngx-fancyindex/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":{"login":"aperezdc","id":723451,"avatar_url":"https://avatars.githubusercontent.com/u/723451?v=3","gravatar_id":"","url":"https://api.github.com/users/aperezdc","html_url":"https://github.com/aperezdc","followers_url":"https://api.github.com/users/aperezdc/followers","following_url":"https://api.github.com/users/aperezdc/following{/other_user}","gists_url":"https://api.github.com/users/aperezdc/gists{/gist_id}","starred_url":"https://api.github.com/users/aperezdc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aperezdc/subscriptions","organizations_url":"https://api.github.com/users/aperezdc/orgs","repos_url":"https://api.github.com/users/aperezdc/repos","events_url":"https://api.github.com/users/aperezdc/events{/privacy}","received_events_url":"https://api.github.com/users/aperezdc/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/aperezdc/ngx-fancyindex/milestones/1","html_url":"https://github.com/aperezdc/ngx-fancyindex/milestones/v0.3.6","labels_url":"https://api.github.com/repos/aperezdc/ngx-fancyindex/milestones/1/labels","id":1254390,"number":1,"title":"v0.3.6","description":null,"creator":{"login":"aperezdc","id":723451,"avatar_url":"https://avatars.githubusercontent.com/u/723451?v=3","gravatar_id":"","url":"https://api.github.com/users/aperezdc","html_url":"https://github.com/aperezdc","followers_url":"https://api.github.com/users/aperezdc/followers","following_url":"https://api.github.com/users/aperezdc/following{/other_user}","gists_url":"https://api.github.com/users/aperezdc/gists{/gist_id}","starred_url":"https://api.github.com/users/aperezdc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aperezdc/subscriptions","organizations_url":"https://api.github.com/users/aperezdc/orgs","repos_url":"https://api.github.com/users/aperezdc/repos","events_url":"https://api.github.com/users/aperezdc/events{/privacy}","received_events_url":"https://api.github.com/users/aperezdc/received_events","type":"User","site_admin":false},"open_issues":2,"closed_issues":0,"state":"open","created_at":"2015-08-13T22:22:16Z","updated_at":"2015-08-13T22:22:33Z","due_on":null,"closed_at":null},"comments":1,"created_at":"2015-08-12T00:52:01Z","updated_at":"2015-08-13T22:23:55Z","closed_at":null,"body":"I have a couple of cases where there are symbolic links for Reasons(TM) but I'd like crawlers to stop seeing them.\r\n\r\nI have a trivial prototype patch:\r\n```\r\n--- ngx_http_fancyindex_module.c.dist\t2015-02-19 04:16:36.000000000 -0800\r\n+++ ngx_http_fancyindex_module.c\t2015-08-05 14:58:51.965238000 -0700\r\n@@ -494,7 +494,7 @@\r\n \r\n         len = ngx_de_namelen(&dir);\r\n \r\n-        if (ngx_de_name(&dir)[0] == '.')\r\n+        if (ngx_de_name(&dir)[0] == '.' || ngx_de_is_link(&dir))\r\n             continue;\r\n \r\n #if NGX_PCRE\r\n```\r\nThis will do for us, but was thinking it might be nice to have it in the upstream version with a configuration control.\r\n\r\nI know nginx itself has an option whether or not to follow symlinks, but that is exactly the opposite of what we want. If it is blocked there, fancyindex will generate href links that result in a 404. "},"comment":{"url":"https://api.github.com/repos/aperezdc/ngx-fancyindex/issues/comments/130865529","html_url":"https://github.com/aperezdc/ngx-fancyindex/issues/27#issuecomment-130865529","issue_url":"https://api.github.com/repos/aperezdc/ngx-fancyindex/issues/27","id":130865529,"user":{"login":"aperezdc","id":723451,"avatar_url":"https://avatars.githubusercontent.com/u/723451?v=3","gravatar_id":"","url":"https://api.github.com/users/aperezdc","html_url":"https://github.com/aperezdc","followers_url":"https://api.github.com/users/aperezdc/followers","following_url":"https://api.github.com/users/aperezdc/following{/other_user}","gists_url":"https://api.github.com/users/aperezdc/gists{/gist_id}","starred_url":"https://api.github.com/users/aperezdc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aperezdc/subscriptions","organizations_url":"https://api.github.com/users/aperezdc/orgs","repos_url":"https://api.github.com/users/aperezdc/repos","events_url":"https://api.github.com/users/aperezdc/events{/privacy}","received_events_url":"https://api.github.com/users/aperezdc/received_events","type":"User","site_admin":false},"created_at":"2015-08-13T22:23:55Z","updated_at":"2015-08-13T22:23:55Z","body":"This seems like an interesting feature to have, I can think of a couple of uses myself  :+1: \r\nI'll be adding a configuration flag for this."}}