Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix exceptions caused by blob.content including \n#149

Merged
buunguyen merged 1 commit into
ovity:masterfrom
norio-nomura:fix-blob-content-may-include-linefeed
Dec 17, 2014
Merged

Fix exceptions caused by blob.content including \n#149
buunguyen merged 1 commit into
ovity:masterfrom
norio-nomura:fix-blob-content-may-include-linefeed

Conversation

@norio-nomura
Copy link
Copy Markdown
Contributor

octotree stopped in some repositories.
e.g.: https://github.com/ReactiveCocoa/ReactiveCocoa

Some blob returns base64 content including \n.
This cause exception in atob() and stop octotree.
The document does not mention about it.
https://developer.github.com/v3/git/blobs/#get-a-blob

octotree stopped in some repositories.
e.g.: https://github.com/ReactiveCocoa/ReactiveCocoa

Some blob returns base64 content including `\n`.
This cause exception in `atob()` and stop octotree.
The document does not mention about it.
https://developer.github.com/v3/git/blobs/#get-a-blob
buunguyen added a commit that referenced this pull request Dec 17, 2014
…e-linefeed

Fix exceptions caused by blob.content including `\n`
@buunguyen buunguyen merged commit 20e16a0 into ovity:master Dec 17, 2014
@buunguyen
Copy link
Copy Markdown
Collaborator

Got you, thanks. Apparently, atob implementation in Chrome and Firefox does the right thing but probably not for Safari.

@buunguyen buunguyen added this to the 1.6 milestone Dec 17, 2014
buunguyen added a commit that referenced this pull request Dec 17, 2014
@norio-nomura norio-nomura deleted the fix-blob-content-may-include-linefeed branch December 18, 2014 00:41
@norio-nomura
Copy link
Copy Markdown
Contributor Author

@buunguyen buunguyen modified the milestones: 1.5.3, 1.6 Mar 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants