{"action":"created","issue":{"url":"https://api.github.com/repos/iainc/Markdown-Content-Blocks/issues/9","repository_url":"https://api.github.com/repos/iainc/Markdown-Content-Blocks","labels_url":"https://api.github.com/repos/iainc/Markdown-Content-Blocks/issues/9/labels{/name}","comments_url":"https://api.github.com/repos/iainc/Markdown-Content-Blocks/issues/9/comments","events_url":"https://api.github.com/repos/iainc/Markdown-Content-Blocks/issues/9/events","html_url":"https://github.com/iainc/Markdown-Content-Blocks/issues/9","id":258807739,"number":9,"title":"Disallow spaces in URLs","user":{"login":"Crissov","id":6200185,"avatar_url":"https://avatars0.githubusercontent.com/u/6200185?v=4","gravatar_id":"","url":"https://api.github.com/users/Crissov","html_url":"https://github.com/Crissov","followers_url":"https://api.github.com/users/Crissov/followers","following_url":"https://api.github.com/users/Crissov/following{/other_user}","gists_url":"https://api.github.com/users/Crissov/gists{/gist_id}","starred_url":"https://api.github.com/users/Crissov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Crissov/subscriptions","organizations_url":"https://api.github.com/users/Crissov/orgs","repos_url":"https://api.github.com/users/Crissov/repos","events_url":"https://api.github.com/users/Crissov/events{/privacy}","received_events_url":"https://api.github.com/users/Crissov/received_events","type":"User","site_admin":false},"labels":[{"id":696791743,"url":"https://api.github.com/repos/iainc/Markdown-Content-Blocks/labels/Suggestion","name":"Suggestion","color":"eeeeee","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2017-09-19T12:47:17Z","updated_at":"2017-09-19T14:13:16Z","closed_at":"2017-09-19T12:55:58Z","author_association":"NONE","body":"You currently have this example among others:\r\n\r\n```\r\n/Savings Account.csv 'Recent Transactions'\r\n```\r\n\r\nTo enable future extensions, literal spaces should be disallowed in file paths, i.e. URLs.\r\n\r\n```\r\n/Savings%20Account.csv 'Recent Transactions'\r\n```\r\n\r\nI think it would be a bad idea to support any alternative escape syntax as found in shells and programming languages, e.g.:\r\n\r\n```\r\n/Savings\\ Account.csv 'Recent Transactions'\r\n/Savings\\_Account.csv 'Recent Transactions'\r\n/Savings_Account.csv 'Recent Transactions'\r\n/Savings&#32;Account.csv 'Recent Transactions'\r\n/Savings&#x20;Account.csv 'Recent Transactions'\r\n(/Savings Account.csv) 'Recent Transactions'\r\n\"/Savings Account.csv\" 'Recent Transactions'\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/iainc/Markdown-Content-Blocks/issues/comments/330552636","html_url":"https://github.com/iainc/Markdown-Content-Blocks/issues/9#issuecomment-330552636","issue_url":"https://api.github.com/repos/iainc/Markdown-Content-Blocks/issues/9","id":330552636,"user":{"login":"Crissov","id":6200185,"avatar_url":"https://avatars0.githubusercontent.com/u/6200185?v=4","gravatar_id":"","url":"https://api.github.com/users/Crissov","html_url":"https://github.com/Crissov","followers_url":"https://api.github.com/users/Crissov/followers","following_url":"https://api.github.com/users/Crissov/following{/other_user}","gists_url":"https://api.github.com/users/Crissov/gists{/gist_id}","starred_url":"https://api.github.com/users/Crissov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Crissov/subscriptions","organizations_url":"https://api.github.com/users/Crissov/orgs","repos_url":"https://api.github.com/users/Crissov/repos","events_url":"https://api.github.com/users/Crissov/events{/privacy}","received_events_url":"https://api.github.com/users/Crissov/received_events","type":"User","site_admin":false},"created_at":"2017-09-19T14:13:16Z","updated_at":"2017-09-19T14:13:16Z","author_association":"NONE","body":"This is deliberately inconsistent with Commonmark's [`link-destination`](http://spec.commonmark.org/0.28/#link-destination) then. CM only allows these:\r\n\r\n```\r\n/Savings%20Account.csv 'Recent Transactions'\r\n</Savings%20Account.csv> 'Recent Transactions'\r\n```\r\n\r\n> A **link destination** consists of either\r\n>\r\n> - a sequence of zero or more characters between an opening `<` and a closing `>` that contains no spaces, line breaks, or unescaped `<` or `>` characters, or\r\n>\r\n> - a nonempty sequence of characters that does not include ASCII space or control characters, and includes parentheses only if (a) they are backslash-escaped or (b) they are part of a balanced pair of unescaped parentheses. (Implementations may impose limits on parentheses nesting to avoid performance issues, but at least three levels of nesting should be supported.)\r\n\r\nYou could try to get spaces allowed inside angle brackets, but I believe this has been discussed and rejected for good reasons.\r\n\r\n```\r\n</Savings Account.csv> 'Recent Transactions'\r\n```\r\n"}}
{"actor":{"display_login":"Crissov"}}