Skip to content
This repository was archived by the owner on Sep 11, 2020. It is now read-only.

remote: sideband support #164

Merged
merged 4 commits into from
Dec 7, 2016
Merged

remote: sideband support #164

merged 4 commits into from
Dec 7, 2016

Conversation

mcuadros
Copy link
Contributor

@mcuadros mcuadros commented Dec 6, 2016

This is the support of Remote for sideband capable servers.

When Remote. Fetch is called If the request contained a side-band-64k or side-band a sideband.Demux is used inside of Remote to read the packfile data.

If the human readable progress is needed a io.Reader/io.Writer should be provided to Repository.Progress field, this pointer will be pass to the remotes and be used when the packfile is read. If the Repository.Progress is not set the no-progress capability is set if supported.

Also a couple of bugs are solved at transport/client:

  • Content-Length is correctly encoded using strconv.Itoa instead of just casting a integer to string.
  • Request encoding is done now using the encoders provide by the `protocol/packp package

I don't know how I ended being impossible to push to the branch, so this is a fork in another org, so is a new PR of #156

remote: Resolving deltas: 100% (32/32), completed with 20 local objects.
remote: Unexpected system error after push was received.
remote: These changes may not be reflected on github.com!
remote: Your unique error code: a3244d2900ca54a5f82ef2504896b2ed

@codecov-io
Copy link

codecov-io commented Dec 6, 2016

Current coverage is 76.91% (diff: 77.55%)

Merging #164 into master will decrease coverage by 0.79%

@@             master       #164   diff @@
==========================================
  Files            88         88          
  Lines          5475       5489    +14   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits           4255       4222    -33   
- Misses          761        815    +54   
+ Partials        459        452     -7   

Powered by Codecov. Last update 3df9063...1ab34c6

@mcuadros mcuadros merged commit 592b538 into src-d:master Dec 7, 2016
mcuadros added a commit that referenced this pull request Jan 31, 2017
* remote: sideband support

* changes
gsalingu-ovhus pushed a commit to gsalingu-ovhus/go-git that referenced this pull request Mar 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants