Skip to content
This repository

Refactored JHttp class that supports CURL, sockets and streams, and a Github library #526

Merged
merged 61 commits into from over 2 years ago

4 participants

ianmacl Jools Andrew Eddie Louis Landry
ianmacl
Owner

Github library currently supports:

  • Pull requests
  • Issues
  • References
  • Gists
LouisLandry and others added some commits
Louis Landry LouisLandry Initial commit of a new HTTP package. 903e1e9
ianmacl ianmacl Added initial work for Github client e9d5d83
ianmacl ianmacl More work on JGithubGists 7e3d991
ianmacl ianmacl More work on JGithubGists 1a8f9cd
ianmacl ianmacl JGithubGists first cut (no comments) ea6153b
ianmacl ianmacl JGithubGists first cut (no comments) 3de57d6
Louis Landry LouisLandry Updating JHttp and transports to support sending data from either an
array or a string.  This should allow more flexibility for various
service requests.
649e5cc
Louis Landry LouisLandry Modify the HTTP package to use the new autoloader conventions. 4b40e4d
ianmacl ianmacl JGithubGists first cut - comments complete 905cb76
ianmacl ianmacl Small changes to return value of gist->create 85bd926
ianmacl ianmacl Start of issues API. Slight adjustment to github base class. 1ec4134
ianmacl ianmacl Post and put were not working - this commit fixes up the issues. 8b1506d
ianmacl ianmacl Began work on Pull Request API
Refactored API connectors to move some common functionality into a base class.
Fixed gist method to return a body in stead of the whole response.
89f577c
ianmacl ianmacl Finished implementation of Pull Requests API.
No pull request comments yet.
1ff6ddf
ianmacl ianmacl Implementation and tests for comments API for pull requests
Fixed code formatting issue with gists tests.
429f445
ianmacl ianmacl Added pull request link into github api a68138e
ianmacl ianmacl Added CA certificate. Not sure if that is the best solution or not. 39b9a8a
ianmacl ianmacl Replaced CA certificate bundle. becbfbf
ianmacl ianmacl Doc block fixes for JGithub 28f21cc
ianmacl ianmacl Doc block fixes for JGithub and JGithubObject 019b192
ianmacl ianmacl Github cleanup e68d1c5
ianmacl ianmacl Moving github library to its own package 26d1099
ianmacl ianmacl Moving files d4fe439
ianmacl ianmacl Changing directory structure 9819cdd
ianmacl ianmacl Refactoring Github library 1b6d507
ianmacl ianmacl Polishing Github library 1448908
ianmacl ianmacl Stabilization after refactoring 85f7ca0
Louis Landry LouisLandry Adding the ability to set options for the HTTP client objects. e8404ca
Louis Landry LouisLandry Adding root certificates to the HTTP package. We'll hook them up soon. f4803d3
Louis Landry LouisLandry Initial update of GitHub package to use the new JHttp package. e422acd
Louis Landry LouisLandry Refactoring some API URL building. 8e9341d
Louis Landry LouisLandry Ensuring that JHttp options are shared amongst transport objects. c53d7c2
Louis Landry LouisLandry Cleaning up the issues, pulls and gists classes for the GitHub package. 014987d
Louis Landry LouisLandry Forgot to have cURL return the response instead of echo it. Oops. b16fd63
Louis Landry LouisLandry Forgot to set the HTTP stream transport to ignore errors so we can
capture them.
8cbd809
ianmacl ianmacl Created unit tests for JGithubPulls c5ee908
ianmacl ianmacl Minor fixes to gists API. 3aa0bb7
ianmacl ianmacl Implemented tests for most of JGithubGists API. 7df2616
ianmacl ianmacl Fixed duplicate test class name issue. 388b770
ianmacl ianmacl Cosmetic changes in tests 7820ae5
ianmacl ianmacl Implement tests for JGithubIssues 2a8185c
ianmacl ianmacl Implemented tests for JGithub. 74fce19
ianmacl ianmacl Fixed code formatting issues. 260e753
ianmacl ianmacl Implemented references API dfca7a3
ianmacl ianmacl Fleshed out implementation of GithubGists::create and edit to support…
… literal data in addition to file names as source data.
16c3e28
ianmacl ianmacl Cleanup leftover stuff. b7654f5
ianmacl ianmacl Finished code coverage for JGithubGists 8c50d3f
ianmacl ianmacl Finished Code Coverage for JGithubRefs 80b553d
ianmacl ianmacl Added test file for GithubGists test. db885b9
ianmacl ianmacl Added refs inferface in JGithub eb8fe0f
ianmacl ianmacl Completed unit test coverage for Github package. 23e9977
ianmacl ianmacl Docblock correction in JHttp 12a7fe2
ianmacl ianmacl Test coverage for the JHttp package. 7d60e38
ianmacl ianmacl Cleaning up old tests. 06b1f14
ianmacl ianmacl Fixes to JHttpTransportSocket e185f8a
ianmacl ianmacl Fixes to tests 8ddb64b
ianmacl ianmacl Updated doc blocks. a83f86b
ianmacl ianmacl Coding style fixes in docblocks of JGithubRefs 017f7fe
ianmacl ianmacl Use cert store. 68216e3
Jools
Collaborator

Build triggered by changes to the base.

Unit testing complete. There were 0 failures and 0 errors from 1828 tests and 10916 assertions.
Checkstyle analysis reported 235 warnings and 0 errors.

ianmacl ianmacl Added content length to JHttp.
Added authentication support to Github.
54d8800
Jools
Collaborator

Build triggered by changes to the head.

Unit testing complete. There were 0 failures and 2 errors from 1831 tests and 10918 assertions.
Checkstyle analysis reported 235 warnings and 0 errors.

Jools
Collaborator

Build triggered by changes to the head.

Unit testing complete. There were 0 failures and 0 errors from 1831 tests and 10926 assertions.
Checkstyle analysis reported 235 warnings and 0 errors.

Jools
Collaborator

Build triggered by changes to the base.

Test log missing. Tests failed to execute.
Checkstyle analysis reported 235 warnings and 0 errors.

Andrew Eddie eddieajau merged commit 53ae34b into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 61 unique commits by 2 authors.

Nov 13, 2011
Louis Landry LouisLandry Initial commit of a new HTTP package. 903e1e9
ianmacl ianmacl Added initial work for Github client e9d5d83
ianmacl ianmacl More work on JGithubGists 7e3d991
ianmacl ianmacl More work on JGithubGists 1a8f9cd
ianmacl ianmacl JGithubGists first cut (no comments) ea6153b
ianmacl ianmacl JGithubGists first cut (no comments) 3de57d6
Louis Landry LouisLandry Updating JHttp and transports to support sending data from either an
array or a string.  This should allow more flexibility for various
service requests.
649e5cc
Louis Landry LouisLandry Modify the HTTP package to use the new autoloader conventions. 4b40e4d
ianmacl ianmacl JGithubGists first cut - comments complete 905cb76
ianmacl ianmacl Small changes to return value of gist->create 85bd926
ianmacl ianmacl Start of issues API. Slight adjustment to github base class. 1ec4134
ianmacl ianmacl Post and put were not working - this commit fixes up the issues. 8b1506d
ianmacl ianmacl Began work on Pull Request API
Refactored API connectors to move some common functionality into a base class.
Fixed gist method to return a body in stead of the whole response.
89f577c
ianmacl ianmacl Finished implementation of Pull Requests API.
No pull request comments yet.
1ff6ddf
ianmacl ianmacl Implementation and tests for comments API for pull requests
Fixed code formatting issue with gists tests.
429f445
ianmacl ianmacl Added pull request link into github api a68138e
ianmacl ianmacl Added CA certificate. Not sure if that is the best solution or not. 39b9a8a
ianmacl ianmacl Replaced CA certificate bundle. becbfbf
ianmacl ianmacl Doc block fixes for JGithub 28f21cc
ianmacl ianmacl Doc block fixes for JGithub and JGithubObject 019b192
ianmacl ianmacl Github cleanup e68d1c5
ianmacl ianmacl Moving github library to its own package 26d1099
ianmacl ianmacl Moving files d4fe439
ianmacl ianmacl Changing directory structure 9819cdd
ianmacl ianmacl Refactoring Github library 1b6d507
ianmacl ianmacl Polishing Github library 1448908
ianmacl ianmacl Stabilization after refactoring 85f7ca0
Louis Landry LouisLandry Adding the ability to set options for the HTTP client objects. e8404ca
Louis Landry LouisLandry Adding root certificates to the HTTP package. We'll hook them up soon. f4803d3
Louis Landry LouisLandry Initial update of GitHub package to use the new JHttp package. e422acd
Louis Landry LouisLandry Refactoring some API URL building. 8e9341d
Louis Landry LouisLandry Ensuring that JHttp options are shared amongst transport objects. c53d7c2
Louis Landry LouisLandry Cleaning up the issues, pulls and gists classes for the GitHub package. 014987d
Louis Landry LouisLandry Forgot to have cURL return the response instead of echo it. Oops. b16fd63
Louis Landry LouisLandry Forgot to set the HTTP stream transport to ignore errors so we can
capture them.
8cbd809
ianmacl ianmacl Created unit tests for JGithubPulls c5ee908
ianmacl ianmacl Minor fixes to gists API. 3aa0bb7
ianmacl ianmacl Implemented tests for most of JGithubGists API. 7df2616
ianmacl ianmacl Fixed duplicate test class name issue. 388b770
ianmacl ianmacl Cosmetic changes in tests 7820ae5
ianmacl ianmacl Implement tests for JGithubIssues 2a8185c
ianmacl ianmacl Implemented tests for JGithub. 74fce19
ianmacl ianmacl Fixed code formatting issues. 260e753
ianmacl ianmacl Implemented references API dfca7a3
ianmacl ianmacl Fleshed out implementation of GithubGists::create and edit to support…
… literal data in addition to file names as source data.
16c3e28
ianmacl ianmacl Cleanup leftover stuff. b7654f5
ianmacl ianmacl Finished code coverage for JGithubGists 8c50d3f
ianmacl ianmacl Finished Code Coverage for JGithubRefs 80b553d
ianmacl ianmacl Added test file for GithubGists test. db885b9
ianmacl ianmacl Added refs inferface in JGithub eb8fe0f
ianmacl ianmacl Completed unit test coverage for Github package. 23e9977
ianmacl ianmacl Docblock correction in JHttp 12a7fe2
ianmacl ianmacl Test coverage for the JHttp package. 7d60e38
ianmacl ianmacl Cleaning up old tests. 06b1f14
ianmacl ianmacl Fixes to JHttpTransportSocket e185f8a
ianmacl ianmacl Fixes to tests 8ddb64b
ianmacl ianmacl Updated doc blocks. a83f86b
ianmacl ianmacl Coding style fixes in docblocks of JGithubRefs 017f7fe
ianmacl ianmacl Use cert store. 68216e3
Nov 15, 2011
ianmacl ianmacl Added content length to JHttp.
Added authentication support to Github.
54d8800
ianmacl ianmacl Fixed errors in curl transport. 146ebc8
Something went wrong with that request. Please try again.